-
-
Notifications
You must be signed in to change notification settings - Fork 698
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
[16.0][OU-ADD] website_payment: Migration to 16.0 #4185
Conversation
/ocabot migration website_payment |
eed1773
to
b68e454
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.
I may be wrong, but I think that this PR needs some changes.
openupgrade_scripts/scripts/website_payment/16.0.1.0/post-migration.py
Outdated
Show resolved
Hide resolved
openupgrade_scripts/scripts/website_payment/16.0.1.0/post-migration.py
Outdated
Show resolved
Hide resolved
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.
@victoralmau please rebase an attend comments.
And other thing: this is not "Migration to 16.0", but "Migration scripts"
openupgrade_scripts/scripts/website_payment/16.0.1.0/post-migration.py
Outdated
Show resolved
Hide resolved
b68e454
to
bf1dd61
Compare
Changes done. |
openupgrade_scripts/scripts/website_payment/16.0.1.0/post-migration.py
Outdated
Show resolved
Hide resolved
bf1dd61
to
44b1c0b
Compare
For getting green CI: /ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 566e8e6. Thanks a lot for contributing to OCA. ❤️ |
Migration to 16.0
@Tecnativa TT43633