Skip to content
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

[MIG] website_sale_hide_empty_category: Migration to 11.0 #256

Merged
merged 3 commits into from
Nov 22, 2018
Merged

[MIG] website_sale_hide_empty_category: Migration to 11.0 #256

merged 3 commits into from
Nov 22, 2018

Conversation

remytms
Copy link
Contributor

@remytms remytms commented Nov 1, 2018

This PR upgrade the website_sale_hide_empty_category from 10.0 to 11.0 and move it from website to e-commerce.

Ted Salmon and others added 2 commits October 31, 2018 22:44
* [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
@remytms
Copy link
Contributor Author

remytms commented Nov 1, 2018

Did I use the right process ? I have cherry-pick all the commit from this PR : OCA/website#530

@pedrobaeza
Copy link
Member

Yes! Thank you very much

@pedrobaeza pedrobaeza added this to the 11.0 milestone Nov 1, 2018
@pedrobaeza pedrobaeza mentioned this pull request Nov 1, 2018
38 tasks
Copy link
Contributor

@simahawk simahawk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 minor issues, approving to not get back ;)

website_sale_hide_empty_category/__manifest__.py Outdated Show resolved Hide resolved
website_sale_hide_empty_category/README.rst Outdated Show resolved Hide resolved
@remytms
Copy link
Contributor Author

remytms commented Nov 6, 2018

@simahawk , @pedrobaeza , Thanks for yours reviews. Things are good now. :)

@remytms
Copy link
Contributor Author

remytms commented Nov 9, 2018

@simahawk , Is it possible to merge this PR or do I need other review before that ? I think I can't do it because I don't see the merge button.

@simahawk
Copy link
Contributor

@pedrobaeza ping :)

@remytms
Copy link
Contributor Author

remytms commented Nov 22, 2018

@pedrobaeza , @simahawk , is it time to merge ? :D

@pedrobaeza pedrobaeza merged commit d89f242 into OCA:11.0 Nov 22, 2018
@pedrobaeza pedrobaeza mentioned this pull request Nov 22, 2018
22 tasks
@remytms
Copy link
Contributor Author

remytms commented Nov 22, 2018

@pedrobaeza Thanks a lot ! :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants