-
-
Notifications
You must be signed in to change notification settings - Fork 58
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
[15.0][MIG] sale_coupon_mass_mailing: Migration to version 15.0 #93
[15.0][MIG] sale_coupon_mass_mailing: Migration to version 15.0 #93
Conversation
…ed 'Mailings' that allows you to link mass-mailing
When renaming, indicate in the last commit such renaming: |
/ocabot migration sale_coupon_mass_mailing |
70f40fc
to
4e60a2c
Compare
OK, I have changed the commit message. Thank you! |
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.
Some minor comments.
Remember to do the PR to OpenUpgrade to rename the module
4e60a2c
to
775d203
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.
Tested. Thanks! 👍
/ocabot merge nobump |
On my way to merge this fine PR! |
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-93-by-pedrobaeza-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
1 similar comment
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-93-by-pedrobaeza-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
I have aborted the CI. You didn't rename the POT file nor updated the content of the files POT and PO. |
775d203
to
5284a2a
Compare
Already updated, thanks |
The POT file is still named |
…rsion 15.0 TT39004
5284a2a
to
96e45da
Compare
sorry, I've already renamed it |
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 1bae319. Thanks a lot for contributing to OCA. ❤️ |
TT39004 @Tecnativa
@victoralmau @chienandalu please review