-
-
Notifications
You must be signed in to change notification settings - Fork 344
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
[14.0][FIX] sale_commission_partial_settlement: handle settlement deletion #575
[14.0][FIX] sale_commission_partial_settlement: handle settlement deletion #575
Conversation
Hi @renda-dev, @PicchiSeba, |
e01b4f9
to
672c126
Compare
Also added migration scripts |
ddef925
to
ab5e010
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.
Functional ok!
To merge with nobump
This commit fixes an issue where, if a settlement was deleted, any partial commission would not be recreated because it was still considered paid. Includes a small refactor of the files and deletion of some unused fields.
ab5e010
to
9b2ece1
Compare
Added a new field "is_fully_settled" |
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 9ee050b. Thanks a lot for contributing to OCA. ❤️ |
This commit fixes an issue where, if a settlement was deleted or canceled, any partial commission would not be recreated because it was still considered paid.
Includes a small refactor of the files and deletion of some unused fields.