-
-
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_criteria_multi_product: Migration to version 15.0 #122
[15.0][MIG] sale_coupon_criteria_multi_product: Migration to version 15.0 #122
Conversation
There's a merge commit, so the history is not correct. You should redo it. |
3c97176
to
52aa17f
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.
Thanks @stefan-tecnativa Some comments below:
- Check tests 🔴
- Reference to the original PR we're superseeding here in the PR description
- Fixup this commit:
|
||
* `Domatix <https://www.domatix.com>`_: | ||
|
||
* Álvaro López Oró |
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.
Let's add @carolina-domatix as contributor
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.
Done
# Copyright 2021 Tecnativa - David Vidal | ||
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). | ||
{ | ||
"name": "Coupons multi product criteria", |
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.
"name": "Coupons multi product criteria", | |
"name": "Coupons multi product criteria in sales", |
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.
Done, thanks for all the suggestions.
52aa17f
to
0f943c4
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.
Great :) Check the tests anyway :)
0f943c4
to
b67d0e8
Compare
b67d0e8
to
239a650
Compare
coupon_criteria_multi_product/migrations/13.0.1.0.0/pre-migration.py
Outdated
Show resolved
Hide resolved
239a650
to
10ef5bd
Compare
10ef5bd
to
ed6a4b5
Compare
ed6a4b5
to
e9c97db
Compare
e9c97db
to
4ac5a18
Compare
/ocabot migration sale_coupon_criteria_multi_product |
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 👍
This PR has the |
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 183cfe8. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT39032
This PR superseeds #88
@chienandalu @pedrobaeza please review!