Skip to content

Conversation

@fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Feb 20, 2025

Current behavior before PR:

It was possible to save a step with step_type "question selection" without any answers. However, this would cause an error when the bot was executed.

Desired behavior after PR is merged:

A constraint has been added to ensure that a "question selection" step cannot be saved without at least one answer.

task-id:4522835


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #198309
Forward-Port-Of: #197104

@robodoo
Copy link
Contributor

robodoo commented Feb 20, 2025

Pull request status dashboard

@fw-bot
Copy link
Contributor Author

fw-bot commented Feb 20, 2025

@akap-odoo @tsm-odoo this PR targets master and is the last of the forward-port chain containing:

To merge the full chain, use

@robodoo r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added the forwardport This PR was created by @fw-bot label Feb 20, 2025
@C3POdoo C3POdoo added the RD research & development, internal work label Feb 20, 2025
@fw-bot
Copy link
Contributor Author

fw-bot commented Feb 20, 2025

@akap-odoo @tsm-odoo ci/runbot failed on this forward-port PR

@akap-odoo
Copy link
Contributor

@robodoo r+

@robodoo
Copy link
Contributor

robodoo commented Feb 21, 2025

@akap-odoo you may want to rebuild or fix this PR as it has failed CI.

Before this commit:

It was possible to save a step with step_type "question selection" without
any answers. However, this would cause an error when the bot was executed.

After this commit:

A constraint has been added to ensure that a "question selection" step cannot be
saved without at least one answer.

X-original-commit: c81d02e
@akap-odoo akap-odoo force-pushed the master-18.0-ensure-question-selection-has-answers-akap-417761-fw branch from 5f4b57f to e7812e1 Compare February 21, 2025 05:00
@fw-bot
Copy link
Contributor Author

fw-bot commented Feb 21, 2025

@akap-odoo @tsm-odoo this PR was modified / updated and has become a normal PR. It must be merged directly.

@akap-odoo
Copy link
Contributor

@robodoo r+

robodoo pushed a commit that referenced this pull request Feb 21, 2025
Before this commit:

It was possible to save a step with step_type "question selection" without
any answers. However, this would cause an error when the bot was executed.

After this commit:

A constraint has been added to ensure that a "question selection" step cannot be
saved without at least one answer.

closes #198707

X-original-commit: c81d02e
Signed-off-by: Matthieu Stockbauer (tsm) <tsm@odoo.com>
Signed-off-by: Akash Parmar (akap) <akap@odoo.com>
@robodoo robodoo added the 18.3 label Feb 21, 2025
@robodoo robodoo closed this Feb 21, 2025
@alexkuhn alexkuhn deleted the master-18.0-ensure-question-selection-has-answers-akap-417761-fw branch February 21, 2025 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

18.3 forwardport This PR was created by @fw-bot RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants