-
-
Notifications
You must be signed in to change notification settings - Fork 675
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][MIG] delivery_total_weight_from_packaging: Migration to 14.0 #900
[14.0][MIG] delivery_total_weight_from_packaging: Migration to 14.0 #900
Conversation
@@ -1,14 +1,14 @@ | |||
# Copyright 2020 Camptocamp SA | |||
# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl) | |||
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl) |
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.
Why this license change?
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.
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, I checked internally and this seems OK. @simahawk are you agree with this change?
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 don't get the reference to OCA/stock-logistics-warehouse#1256 🤔
There's no license mix here AFAIS (on v13 at least).
Also, "hiding" the license change in the same commit IMO it's not that good.
If there's no special / good reason to make it AGPL I'd roll back this change
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.
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 PR has been rebased and updated. 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.
Runbot was red, I restarted it to check the logs
Change the way the weight is computed on move, picking and package to include packaging weight instead of the weight of the product only.
e20dcea
to
9abaeb9
Compare
there's a problem w/ runbot not related to this PR. Maybe related to OCA/maintainer-quality-tools#697 /ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 13c9bc4. Thanks a lot for contributing to OCA. ❤️ |
No description provided.