-
Notifications
You must be signed in to change notification settings - Fork 6
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
[ADD] sustainability_purchase: product's supplier emission factor preceding order #223
Conversation
ac505b0
to
ac9a00f
Compare
f0e2310
to
a8b461c
Compare
eff8781
to
ef28968
Compare
… and account move
ef28968
to
58be404
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.
Ok but we'll need to refactor _compute_carbon_debt because on large database it will cause performance issues.
@@ -98,6 +119,14 @@ def _compute_carbon_is_date_locked(self): | |||
# -------------------------------------------- | |||
|
|||
@api.depends( | |||
# Seller |
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.
This is not optimized because it triggers recompute on all account.move.line when you change your seller_ids or the product. Same for company lock date.
/ocabot merge minor |
Hey, thanks for contributing! Proceeding to merge this for you. |
…rder (#223) - #201 --------- Co-authored-by: Adam Bonnet <[email protected]>
Related Issues