-
-
Notifications
You must be signed in to change notification settings - Fork 697
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] stock_picking_return_refund_option: Migration to 15.0 #1204
[15.0][MIG] stock_picking_return_refund_option: Migration to 15.0 #1204
Conversation
…fund options in picking directly
…ntities when user check to refund in operation lines.
…crash test due to this Odoo commit OCA/OCB@8ca10a8
We force the delivery creating stock.move.line for avoiding circumstances where the assignation is not done. The SO creation is also simplified using Form utility, and now as we don't depend on the warehouse, we remove such code as well.
There's still some cases that makes the pickings to not be completed, so we force them and also switch to Form for simplifying code.
2b2ee37
to
fd0c002
Compare
@sergio-teruel please review |
@pedrobaeza please review |
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.
Functional and code review
/ocabot migration stock_picking_return_refund_option |
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 + tested on runbot
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at c6d4b59. Thanks a lot for contributing to OCA. ❤️ |
@Tecnativa TT36568