-
-
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_multi_gift: Migration to 15.0 #98
[15.0][MIG] sale_coupon_multi_gift: Migration to 15.0 #98
Conversation
This must be renamed to |
It depends on |
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.
Code review OK
But should we split it to separate the multi_gift definition from the sales part? |
Now we use the product field in the multi gift line as a mask for a many2many field that can be used by a third module as a way to offer optional gifts to the user. TT30850
When we've got mixed optional rewards we need to tell for sure which reward line belongs a sale line to so we can apply the right product when the reward is updated. TT34266
b976dea
to
cc662c8
Compare
Done |
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.
👍 Code review
/ocabot migration sale_coupon_multi_gift |
cc662c8
to
a15546f
Compare
Changes done |
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.
/ocabot merge nobump
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 961c0db. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT39005
please review @CarlosRoca13 @victoralmau