-
-
Notifications
You must be signed in to change notification settings - Fork 149
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] portal_odoo_debranding: Migration to 17.0 #78
[17.0][MIG] portal_odoo_debranding: Migration to 17.0 #78
Conversation
/ocabot migration portal_odoo_debranding |
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.
Thanks @NICO-SOLUTIONS for porting this module.
Some minor comment related to XML tag not necessary anymore in index.html raising a WARNING during installation.
Apart from this, 👍
b3315e8
to
ea97407
Compare
ea97407
to
11fbb40
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.
easy code review.
Functional test
LGTM
Hi @legalsylvain, is it possible to get this one merged? |
[UPD] Update portal_odoo_debranding.pot [UPD] README.rst [UPD] README.rst
11fbb40
to
f2331ef
Compare
f2331ef
to
cef2e1a
Compare
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at bdf76a9. Thanks a lot for contributing to OCA. ❤️ |
@yvaucher thanks! |
Standard Migration to 17.0