-
-
Notifications
You must be signed in to change notification settings - Fork 740
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
base: 18.0
Are you sure you want to change the base?
[18.0][MIG] stock_product_qty_by_packaging #2288
Conversation
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/
/ocabot migration stock_product_qty_by_packaging |
2564b16
to
c4f6d5b
Compare
@@ -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" |
There was a problem hiding this comment.
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" |
There was a problem hiding this comment.
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
_qty_by_pkg__qty_field_name = "product_uom_qty" | |
_qty_by_pkg__qty_field_name = "product_qty" |
There was a problem hiding this comment.
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
No description provided.