-
-
Notifications
You must be signed in to change notification settings - Fork 501
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
[18.0][MIG] website_sale_hide_empty_category: Migration to version 18.0 #1011
[18.0][MIG] website_sale_hide_empty_category: Migration to version 18.0 #1011
Conversation
* [ADD] website_sale_category_empty: Create module * Add module to hide categories that have no products associated with them * [FIX] website_sale_hide_empty_category: PR Fix * Add trailing comma to manifest * Rename Module
Currently translated at 100.0% (2 of 2 strings) Translation: e-commerce-13.0/e-commerce-13.0-website_sale_hide_empty_category Translate-URL: https://translation.odoo-community.org/projects/e-commerce-13-0/e-commerce-13-0-website_sale_hide_empty_category/nl/
Currently translated at 100.0% (2 of 2 strings) Translation: e-commerce-13.0/e-commerce-13.0-website_sale_hide_empty_category Translate-URL: https://translation.odoo-community.org/projects/e-commerce-13-0/e-commerce-13-0-website_sale_hide_empty_category/ca/
Currently translated at 100.0% (2 of 2 strings) Translation: e-commerce-13.0/e-commerce-13.0-website_sale_hide_empty_category Translate-URL: https://translation.odoo-community.org/projects/e-commerce-13-0/e-commerce-13-0-website_sale_hide_empty_category/es/
- Create setUpClass method if necessary - Switch to setUpClass for avoiding repeat the same setup for each test. - Include context keys for avoiding mail operations overhead.
/ocabot migration website_sale_hide_empty_category |
75afd32
to
8efc1cd
Compare
8efc1cd
to
c53530c
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.
LGTM 👍
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.
Working fine 👍
This PR has the |
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 7586bc0. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT54389
@CarlosRoca13 @chienandalu please review