-
-
Notifications
You must be signed in to change notification settings - Fork 166
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[16.0][MIG] event_registration_multi_qty: Migration to version 16.0 #318
[16.0][MIG] event_registration_multi_qty: Migration to version 16.0 #318
Conversation
Currently translated at 100.0% (7 of 7 strings) Translation: event-10.0/event-10.0-event_registration_multi_qty Translate-URL: https://translation.odoo-community.org/projects/event-10-0/event-10-0-event_registration_multi_qty/es/
After odoo/odoo@a4d50b4, the passed value is not taken into account when a default value is set on the field. This adds the option of passing the desired value through the context and force that update.
If we force the multi qty mode by default there's a high chance to come across unexpected CI errors.
Currently translated at 66.6% (6 of 9 strings) Translation: event-14.0/event-14.0-event_registration_multi_qty Translate-URL: https://translation.odoo-community.org/projects/event-14-0/event-14-0-event_registration_multi_qty/it/
cb77041
to
c69200a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test and code review 👍 . Tiny suggestion:
c69200a
to
2d60120
Compare
please review @CarlosRoca13 |
This PR has the |
/ocabot migration event_registration_multi_qty |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at e091ca8. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT43666
@chienandalu please review!