Skip to content

[17.0] [MIG] sale_margin_delivered_security#257

Open
rrebollo wants to merge 49 commits intoOCA:17.0from
BinhexTeam:17.0-mig-sale_margin_delivered_security
Open

[17.0] [MIG] sale_margin_delivered_security#257
rrebollo wants to merge 49 commits intoOCA:17.0from
BinhexTeam:17.0-mig-sale_margin_delivered_security

Conversation

@rrebollo
Copy link

Standard migration, backport from v18.0

Depends on:

#187
@BinhexTeam T19998

chienandalu and others added 30 commits December 2, 2025 10:00
…l on delivery purchase price

[UPD] README.rst

[UPD] Update sale_margin_delivered.pot

Update translation files

Updated by Update PO files to match POT (msgmerge) hook in Weblate.
…not delivered qty (OCA#43)

[FIX] sale_margin_delivered: Set % delivered percent to quantities ordered if the line has not delivered quantities

[ADD] icon.png
…to compute delivered margin. Other module tests fails

[UPD] README.rst
We don't want to compute margin based on stock moves for non storable
products. This specially true for services and kits.

TT33499
Currently translated at 100.0% (8 of 8 strings)

Translation: margin-analysis-16.0/margin-analysis-16.0-sale_margin_delivered
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-16-0/margin-analysis-16-0-sale_margin_delivered/es/
- Include context keys for avoiding mail operations overhead.
Currently translated at 100.0% (8 of 8 strings)

Translation: margin-analysis-16.0/margin-analysis-16.0-sale_margin_delivered
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-16-0/margin-analysis-16-0-sale_margin_delivered/it/
Currently translated at 100.0% (11 of 11 strings)

Translation: margin-analysis-16.0/margin-analysis-16.0-sale_margin_delivered
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-16-0/margin-analysis-16-0-sale_margin_delivered/it/
'price_reduce' is deprecated and removed into the next version. Compute
the prirce_reduct from the price_subotal / product_uom_qty. We might
be tempted to use the 'price_reduce_taxecl' field from the sale order
line but this field is rounded by default to the monetary precision.

As an additional benefit this change ensures the compatibility with
the 'sale_triple_discount' addon. Indeed, when
'sale_triple_discount' is installed, the discount field is not used
as an aggregation of all the applied discount. It's only use to store
the first discount applied. Therefore, the field is not properly
computed since it doesn't include the second and third discount.
Put all the new fields after the orginal margin fields from odoo. (prior to this change, the new fields were displayed among the margin fields from Odoo
lmignon and others added 19 commits December 2, 2025 10:00
…ion of 1

As it's done into the sale_margin addon from odoo
Currently translated at 100.0% (11 of 11 strings)

Translation: margin-analysis-16.0/margin-analysis-16.0-sale_margin_delivered
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-16-0/margin-analysis-16-0-sale_margin_delivered/it/
Following OCA/product-attribute#1538, the security is now done via mixin and the view can be removed.

@moduon MT-5158
Currently translated at 100.0% (5 of 5 strings)

Translation: margin-analysis-16.0/margin-analysis-16.0-sale_margin_delivered_security
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-16-0/margin-analysis-16-0-sale_margin_delivered_security/it/
…elds

The fields to secure are defined on the sale.order.line model not on sale.order
Currently translated at 100.0% (8 of 8 strings)

Translation: margin-analysis-16.0/margin-analysis-16.0-sale_margin_delivered_security
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-16-0/margin-analysis-16-0-sale_margin_delivered_security/it/
@rrebollo rrebollo marked this pull request as ready for review February 28, 2026 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.