-
-
Notifications
You must be signed in to change notification settings - Fork 686
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][MIG] sale_line_refund_to_invoice_qty: Migration to 16.0 #1461
[16.0][MIG] sale_line_refund_to_invoice_qty: Migration to 16.0 #1461
Conversation
e357d5a
to
dd1a67f
Compare
dd1a67f
to
252adfa
Compare
fbea4b5
to
1cea083
Compare
1034b59
to
0c9f2d0
Compare
…tion for usability and only display on credit notes
when you mark a refund to not be reinvoiced, the invoice qty should not be reduced, because it is already invoiced and you won't invoice it again.
0c9f2d0
to
4502e3c
Compare
/ocabot migration sale_line_refund_to_invoice_qty |
This PR has the |
@AaronHForgeFlow |
@BT-jmichaud this LGTM but there is an open question in this PR: #1383 that is forward ported here. I prefer to wait until that question is solved or until @RodrigoBM reviews this one. |
Well, as long as @RodrigoBM approved this one I think the question is solved. |
1 similar comment
Well, as long as @RodrigoBM approved this one I think the question is solved. |
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at e1a0aa5. Thanks a lot for contributing to OCA. ❤️ |
#1250
Add fix: #1383