-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[17.0][MIG] web_widget_domain_editor_dialog: Migration to 17.0 #2875
[17.0][MIG] web_widget_domain_editor_dialog: Migration to 17.0 #2875
Conversation
When clicking edit the list of record an error happened due to lack of proper initialization of the default domain.
- IMP: Now it's possible to work with the domain editor when the widget has de in_dialog option. - FIX: When a boolean opertator was present in the domain, the editor dialog would raise an error as the default filter couldn't be interpretated. TT32827
The dialog needs to keep the context so translations and other things work as expected. TT45543
LGTM 👍 Thanks 😄 |
/ocabot migration web_widget_domain_editor_dialog |
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.
Tested in runboat
/ocabot merge nobump
On my way to merge this fine PR! |
Congratulations, your PR was merged at eb17de2. Thanks a lot for contributing to OCA. ❤️ |
Standard migration
cc @Tecnativa TT49927
please review @CarlosRoca13