Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][MIG] product_weighable_default_weight: Migration to 16.0 (from 12.0) #1746

Open
wants to merge 5 commits into
base: 16.0
Choose a base branch
from

Conversation

legalsylvain
Copy link
Contributor

@legalsylvain legalsylvain commented Oct 20, 2024

Trivial migration.

Description

This module extends the functionality of product module to set
default value on weight field for weighable products.

Once installed, when a user create a new weighable product
(uom_id.measure_type == "weight"), the weight field
will be computed, depending on the UoM.

  • if UoM is "kg", weight will be 1.0
  • if UoM is "ton", weight will be 1000.0

This module is interesting if you use other modules based on the
weight field. (Ex: sale_order_weight in OCA/sale-workflow repo)

@legalsylvain legalsylvain force-pushed the 16.0-mig-product_weighable_default_weight branch from a5583ea to faefdbd Compare November 5, 2024 16:48
@legalsylvain legalsylvain marked this pull request as ready for review November 5, 2024 16:49
@jbaudoux
Copy link
Contributor

jbaudoux commented Nov 6, 2024

/ocabot migration product_weighable_default_weight

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Nov 6, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 6, 2024
70 tasks
@jbaudoux
Copy link
Contributor

jbaudoux commented Nov 6, 2024

@legalsylvain Can you fix pre-commit?

@legalsylvain legalsylvain force-pushed the 16.0-mig-product_weighable_default_weight branch from faefdbd to 95103d8 Compare November 6, 2024 10:34
@legalsylvain
Copy link
Contributor Author

@legalsylvain Can you fix pre-commit?

thanks. done.

@jbaudoux
Copy link
Contributor

jbaudoux commented Nov 6, 2024

Test failing due to odoo/odoo#186365

@legalsylvain
Copy link
Contributor Author

Test failing due to odoo/odoo#186365

thanks a lot for the investigation. Let's wait what odoo is doing.

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.

3 participants