-
-
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
[17.0] [ADD] sale_blanket_order_tier_validation #3277
base: 17.0
Are you sure you want to change the base?
[17.0] [ADD] sale_blanket_order_tier_validation #3277
Conversation
4903a4b
to
a61d983
Compare
def action_confirm(self): | ||
"""Set the state to 'Open' when clicking confirmed instead of during compute.""" | ||
for rec in self: | ||
rec.write({"state": "open"}) |
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.
FYI I did OCA/server-ux#939 so that tier validations can be applied to computed state fields as in this model.
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.
Would you consider reviewing that change and refactoring this module to work with it? You wouldn't have to jump through hoops like you do in the code above.
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.
I did the refactoring in ursais#14. What do you think?
@Nikul-OSI Could you rebase ? |
a61d983
to
1ff1f88
Compare
Done |
Depends on
sale_blanket_order [17.0][MIG] sale_blanket_order #3255
base_tier_validation https://github.com/OCA/server-ux/tree/17.0/base_tier_validation