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

[18.0][MIG] stock_product_qty_by_packaging #2288

Open
wants to merge 29 commits into
base: 18.0
Choose a base branch
from

Conversation

sebalix
Copy link
Contributor

@sebalix sebalix commented Mar 11, 2025

No description provided.

simahawk and others added 27 commits March 11, 2025 16:52
Currently translated at 100.0% (4 of 4 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_product_qty_by_packaging
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_product_qty_by_packaging/it/
Currently translated at 100.0% (4 of 4 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_product_qty_by_packaging
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_product_qty_by_packaging/it/
Currently translated at 100.0% (4 of 4 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_product_qty_by_packaging
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_product_qty_by_packaging/it/
Add product_qty_by_packaging_display field to view_stock_move_line_detailed_operation_tree view
Currently translated at 100.0% (4 of 4 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_product_qty_by_packaging
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_product_qty_by_packaging/fr/
@sebalix sebalix added this to the 18.0 milestone Mar 11, 2025
@sebalix
Copy link
Contributor Author

sebalix commented Mar 11, 2025

/ocabot migration stock_product_qty_by_packaging

@OCA-git-bot OCA-git-bot mentioned this pull request Mar 11, 2025
40 tasks
@sebalix sebalix force-pushed the 18-mig-stock_product_qty_by_packaging branch from 2564b16 to c4f6d5b Compare March 11, 2025 16:49
@sebalix sebalix requested review from simahawk and jbaudoux March 11, 2025 16:51
@sebalix sebalix marked this pull request as ready for review March 11, 2025 16:51
@@ -10,4 +10,4 @@ class StockMoveLine(models.Model):
_name = "stock.move.line"
_inherit = ["stock.move.line", "product.qty_by_packaging.mixin"]

_qty_by_pkg__qty_field_name = "reserved_qty"
_qty_by_pkg__qty_field_name = "quantity_product_uom"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mapping of stock.move.line fields before/after 17.0 from what I see:

  • reserved_qty => quantity_product_uom
  • reserved_uom_qty => quantity

_name = "stock.move"
_inherit = ["stock.move", "product.qty_by_packaging.mixin"]

_qty_by_pkg__qty_field_name = "product_uom_qty"
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't it be in the uom of the product ? Like for the move line

Suggested change
_qty_by_pkg__qty_field_name = "product_uom_qty"
_qty_by_pkg__qty_field_name = "product_qty"

Copy link
Contributor Author

@sebalix sebalix Mar 12, 2025

Choose a reason for hiding this comment

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

product_uom_qty and product_qty didn't changed between 16.0 and 18.0, but that means it's a bug even in 16.0? And should be fixed in its own commit

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.

10 participants