Code review comment for lp:~camptocamp/stock-logistic-warehouse/7.0-port-base_product_merge

Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

just curious: why do we need the _columns in BaseProductMerge for 7.0 and not in the original code?

More importantly: are we sure about this:
"""All the one2many relations (invoices, sale_orders, ...) are updated in order to link to the new resource."""

It sounds very very dangerous to me. Or maybe this is only for some of the targeted invoices and sale orders?

review: Needs Information

« Back to merge proposal