Skip to content

Conversation

@dreispt
Copy link
Member

@dreispt dreispt commented Jan 4, 2026

chienandalu and others added 29 commits January 4, 2026 16:07
In sale_margin, when the line already exists, the purchase_price
is computed with the product onchange method, so if the user doesn't
have the field available in the view it's not going to get the right
price. This should be fixed in v13 with the new compute fields
possibilities.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: margin-analysis-15.0/margin-analysis-15.0-sale_margin_security
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-15-0/margin-analysis-15-0-sale_margin_security/
Currently translated at 100.0% (3 of 3 strings)

Translation: margin-analysis-16.0/margin-analysis-16.0-sale_margin_security
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-16-0/margin-analysis-16-0-sale_margin_security/es/
Set group_sale_margin_security in all sensible fields.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: margin-analysis-16.0/margin-analysis-16.0-sale_margin_security
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-16-0/margin-analysis-16-0-sale_margin_security/
Currently translated at 85.7% (6 of 7 strings)

Translation: margin-analysis-16.0/margin-analysis-16.0-sale_margin_security
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-16-0/margin-analysis-16-0-sale_margin_security/es/
Both modules do essentially the same thing: apply a restriction over product costs. Thus, it makes sense to share the same permission groups.

This refactor, that depends on OCA/product-attribute#1538, improves the module readme and makes `sale_margin_security` auto-installable addon when `product_cost_security` is found.

A migration script is provided to make sure the same users still retain the same permissions.

@moduon MT-5158
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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

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

Translation: margin-analysis-18.0/margin-analysis-18.0-sale_margin_security
Translate-URL: https://translation.odoo-community.org/projects/margin-analysis-18-0/margin-analysis-18-0-sale_margin_security/tr/
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.