[18.0][MIG] delivery_package_fee: Migration to 18.0 #921
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
A list of Package Fees can be added on shipping methods. When a outgoing transfer is done, for each package fee configured on the shipping method, a new sale order line is created with: * The product selected on the Package Fee * The product name with the number of the transfer in the line's description (e.g. "Service Fee (WH/OUT/00036)") * The quantity equal to the number of packages in the transfer * The unit price equal to the price set on the product's pricelist (so it can be different per customer and even have different pricing depending on the number of packages) * The taxes configured on the product, fiscal position applies if any. Package Fee lines are added only if their quantity and price is above zero.
Flag package fee order lines with a new boolean 'is_delivery_package_fee'. Standard method '_is_delivery' will also consider this field to exclude the lines from the computation of the 'expected_date' in 'sale' module.
'order_line' can be empty when SO is copied from direct call to copy with False as default.
Currently translated at 100.0% (19 of 19 strings) Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_package_fee Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_package_fee/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_package_fee Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_package_fee/
Currently translated at 100.0% (19 of 19 strings) Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_package_fee Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_package_fee/it/
Currently translated at 89.4% (17 of 19 strings) Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_package_fee Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_package_fee/fr/
jbaudoux
reviewed
Dec 30, 2024
Comment on lines
3
to
4
| A list of Package Fees can be added on shipping methods. When a outgoing | ||
| transfer is done, for each package fee configured on the shipping |
There was a problem hiding this comment.
Suggested change
| A list of Package Fees can be added on shipping methods. When a outgoing | |
| transfer is done, for each package fee configured on the shipping | |
| A list of Package Fees can be configured on delivery methods. When a outgoing | |
| transfer is done, for each package fee configured on the delivery |
delivery_package_fee/__manifest__.py
Outdated
| # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). | ||
| { | ||
| "name": "Delivery Package Fees", | ||
| "summary": "Add fees on delivered packages on shipping methods", |
There was a problem hiding this comment.
Suggested change
| "summary": "Add fees on delivered packages on shipping methods", | |
| "summary": "Add fees on sales order for delivered packages", |
|
|
||
| A list of Package Fees can be added on shipping methods. When a outgoing | ||
| transfer is done, for each package fee configured on the shipping | ||
| method, a new sale order line is created with: |
There was a problem hiding this comment.
Suggested change
| method, a new sale order line is created with: | |
| method, a new sales order line is created with: |
chaule97
force-pushed
the
18.0-mig-delivery_package_fee
branch
from
1b2e553 to
ecfeffd
Compare
jbaudoux
approved these changes
Dec 31, 2024
ajaniszewska-dev
approved these changes
Dec 31, 2024
|
/ocabot migration delivery_package_fee |
|
/ocabot merge nobump delivery_package_fee |
|
Hi @jbaudoux. Your command failed:
Ocabot commands
More information
|
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at f55679d. Thanks a lot for contributing to OCA. ❤️ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
is_storablewas introduced for simpler tracking instead of typeproduct, in odoo/odoo@83d0f2b.