Skip to content

[17.0][IMP] l10n_eu_product_adr_dangerous_goods#251

Open
edescalona wants to merge 2 commits into
OCA:17.0from
BinhexTeam:17.0-imp-l10n_eu_product_adr_dangerous_goods
Open

[17.0][IMP] l10n_eu_product_adr_dangerous_goods#251
edescalona wants to merge 2 commits into
OCA:17.0from
BinhexTeam:17.0-imp-l10n_eu_product_adr_dangerous_goods

Conversation

@edescalona
Copy link
Copy Markdown

@edescalona edescalona commented Dec 2, 2025

@BinhexTeam

Allows you to display the Adr factor value in the Product Variants view and modify this value if necessary.

@edescalona
Copy link
Copy Markdown
Author

ping @rrebollo

@edescalona edescalona marked this pull request as ready for review December 2, 2025 21:12
Copy link
Copy Markdown

@rrebollo rrebollo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, answer my observations.

Comment thread l10n_eu_product_adr/models/product_product.py
Copy link
Copy Markdown

@rrebollo rrebollo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review. LGTM!

@rrebollo
Copy link
Copy Markdown

@edescalona we need help to speed up the approval of the migration PR.

@edescalona edescalona force-pushed the 17.0-imp-l10n_eu_product_adr_dangerous_goods branch from f69cd41 to f7a134d Compare March 23, 2026 23:01
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.

2 participants