-
-
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
[12.0][MIG] - Wildcard in advanced migration to 12.0 #1326
[12.0][MIG] - Wildcard in advanced migration to 12.0 #1326
Conversation
In order to get visibility on https://www.odoo.com/apps the OCA board has decided to add the OCA as author of all the addons maintained as part of the association.
Conflicts: web_advanced_search_wildcard/__openerp__.py
[MOD] Update manifest with latest OCA standard Conflicts: web_advanced_search_wildcard/__openerp__.py
[MOD] README [ADD] Screenshot
…h' (OCA#1142) * We keep the historic feature *matches* provided by web_advanced_search_wildcard, but we add *starts with* and *ends with* with are super-easy and intuitive to use and cover the majority of use cases. * FR translation
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
@gva-acsone, can you review this please. |
d9a1b2d
to
74616f7
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.
+1 Functional test
Shouldn't this be renamed to accommodate new features? And also as the search feature is now called "Custom filter", this can be confused with existing OCA module that adds a new feature |
+1 |
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.
LGTM 👍, code review
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.
Works great, some recommendations.
This PR has the |
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.
go
merge! |
@sbejaoui : could you take a look on the @daramousk remarks ? If you have no time, I propose to merge this PR, and wait for another one regarding @daramousk remarks that seems to be cosmectics / non blocking remarks. thanks. |
Done |
/ocabot merge |
On my way to merge this fine PR! |
Congratulations, your PR was merged at fafbbe8. Thanks a lot for contributing to OCA. ❤️ |
In addition to the migration this PR add two new options to web_advanced_search_wildcard: