Code review comment for lp:~camptocamp/stock-logistic-warehouse/add-stock_ownership_nbi

Revision history for this message
Romain Deheele - Camptocamp (romaindeheele) wrote :

Guewen,

Changes about sale and sale order line (_prepare methods on sale order, field location_id on sale order line, sale views and sale tests)
are already in sale_sourced_by_line addon proposed here :
https://code.launchpad.net/~camptocamp/sale-wkfl/7.0-add-sale_sourced_and_stock_ownership/+merge/194527

It seems to me that the best location is sale_sourced_by_line because pointed changes concern sale objects.

What do you think about it?

Regards,

Romain

review: Needs Information (code review)

« Back to merge proposal