Skip to content

[19.0][MIG] product_packaging_dimension: Migration to 19.0#2327

Open
yankinmax wants to merge 20 commits into
OCA:19.0from
camptocamp:19.0-mig-product_packaging_dimension
Open

[19.0][MIG] product_packaging_dimension: Migration to 19.0#2327
yankinmax wants to merge 20 commits into
OCA:19.0from
camptocamp:19.0-mig-product_packaging_dimension

Conversation

@yankinmax

Copy link
Copy Markdown
Contributor
  • Added new dependency on product_packaging along with product_logistics_uom to align with the 19.0 module split.

  • Kept product_packaging_dimension focused on dimensions only: it now adds packaging_length, width, height, and the length UoM label.

  • Removed duplicated ownership of fields now provided by product_packaging: weight, volume, weight_uom_name, and volume_uom_name.

  • Updated _compute_volume() as an override with super(), so dimensional volume calculation extends the base packaging behavior.

  • Added a pre-migration script to convert legacy per-record length_uom_id and weight_uom_id values into the configured global logistics UoMs before the old columns disappear.

  • Updated views and tests to use the new base packaging view structure and global UoM configuration, without changing the module’s business purpose.

Depends on:

@OCA-git-bot OCA-git-bot added series:19.0 mod:product_packaging_dimension Module product_packaging_dimension labels Jun 16, 2026
@yankinmax yankinmax force-pushed the 19.0-mig-product_packaging_dimension branch 2 times, most recently from e50fc5f to 6c86f51 Compare June 17, 2026 08:33
Comment thread product_packaging_dimension/models/product_packaging.py Outdated
Comment thread product_packaging_dimension/models/product_packaging.py Outdated
Comment thread product_packaging_dimension/__manifest__.py Outdated
Comment thread product_packaging_dimension/views/product_packaging.xml
@yankinmax yankinmax force-pushed the 19.0-mig-product_packaging_dimension branch from 6c86f51 to 327e7b9 Compare June 17, 2026 12:36

@ivantodorovich ivantodorovich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!!

@imlopes imlopes left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@BhaveshHeliconia BhaveshHeliconia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.