-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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][IMP] sale_force_invoiced: update untaxed_amount_to_invoice #3101
Conversation
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 remarks inline.
Otherwise, LGTM.
However, I don't use this module. I prefer original authors confirm this change is correct.
CC : @JordiBForgeFlow
65c47d8
to
6a1010e
Compare
Recompute untaxed_amount_to_invoice on the sale order lines when force_invoiced is changed.
6a1010e
to
431d427
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!
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
LGTM
thanks
This PR has the |
/ocabot merge patch |
On my way to merge this fine PR! |
Congratulations, your PR was merged at e950113. Thanks a lot for contributing to OCA. ❤️ |
Recompute untaxed_amount_to_invoice on the sale order lines when force_invoiced is changed.
Issue #3093