-
-
Notifications
You must be signed in to change notification settings - Fork 980
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] sale_delivery_state #1851
Conversation
Make it optional to work with module `delivery` without adding it as a dependency
Co-authored-by: Simone Orsi <simahawk@users.noreply.github.com>
f3ba9cb
to
99dd27b
Compare
somebody can help me review this PR? |
99dd27b
to
1bd0d2c
Compare
/ocabot migration sale_delivery_state |
order.delivery_state = "no" | ||
elif order.force_delivery_state or order._all_qty_delivered(): | ||
order.delivery_state = "done" | ||
elif order._partially_delivered(): |
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.
Checking in the same loop would be more efficient than filtering, getting precision and going through the lines for both the _all_qty_delivered method and the _partially_delivered method, but I suppose it will be because it is necessary to extend these methods in some case ...
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 92c235c. Thanks a lot for contributing to OCA. ❤️ |
No description provided.