[17.0][OU-ADD] stock: migrate to 17.0#4557
Conversation
|
/ocabot migration stock Depends on :
|
|
for reference the 4 key commits for the stock migration to v17: https://github.com/akretion/odoo-module-diff-analysis/tree/main/17.0/stock |
acpMicrocom
force-pushed
the
17.0_ou_stock
branch
from
f85f024 to
ebe7b30
Compare
acpMicrocom
force-pushed
the
17.0_ou_stock
branch
from
ebe7b30 to
9febc4b
Compare
hbrunn
left a comment
hbrunn
left a comment
There was a problem hiding this comment.
@acpMicrocom I haven't seen followup on this PR lately, so I'll implement my comments in a new PR
|
|
||
| stock / stock.move / scrap_id (many2one) : NEW relation: stock.scrap | ||
| stock / stock.move / scrap_ids (one2many) : DEL relation: stock.scrap | ||
| # NOTHING TO DO |
There was a problem hiding this comment.
this is done in pre-migration where you transform stock.scrap#move_id
| @@ -0,0 +1,19 @@ | |||
| # Copyright 2024 Viindoo Technology Joint Stock Company (Viindoo) | |||
There was a problem hiding this comment.
your copyright notices look wrong
|
|
||
| _field_renames = [ | ||
| ("stock.move", "stock_move", "quantity_done", "quantity"), | ||
| ("stock.move.line", "stock_move_line", "qty_done", "quantity"), |
There was a problem hiding this comment.
it's more complex than that. as this field now as well counts what is done and what it reserved, you need to add reserved_qty if qty_done was 0 or the move's state isn't done (depends a bit on how users used v16).
I suggest to copy the column instead of renaming to allow subsequent code make its own choices here
| stock / stock.move.line / picked (boolean) : NEW hasdefault: compute | ||
| stock / stock.move.line / quant_id (many2one) : NEW relation: stock.quant | ||
| stock / stock.move.line / quantity_product_uom (float) : NEW isfunction: function, stored | ||
| stock / stock.move.line / reserved_qty (float) : DEL |
| stock / stock.picking / picking_properties (properties): NEW hasdefault: compute | ||
| stock / stock.picking / rating_ids (one2many) : NEW relation: rating.rating | ||
| stock / stock.picking / return_id (many2one) : NEW relation: stock.picking | ||
| stock / stock.picking / return_ids (one2many) : NEW relation: stock.picking |
There was a problem hiding this comment.
those we'll need to reconstruct from stock.move#origin_returned_move_id
| stock / stock.picking.type / auto_print_reception_report (boolean): NEW | ||
| stock / stock.picking.type / auto_print_reception_report_labels (boolean): NEW | ||
| stock / stock.picking.type / auto_print_return_slip (boolean): NEW | ||
| stock / stock.picking.type / default_location_return_id (many2one): NEW relation: stock.location |
There was a problem hiding this comment.
this can be calculated that from the warehouse's old return picking type's default location
|
superseded by #4681 |
6 participants
No description provided.