Sales - Workflow and Organization

Code review comment for lp:~camptocamp/sale-wkfl/7.0-add-sale_sourced_and_stock_ownership

  1. 7.0-add-sale_sourced_and_stock_ownership
  2. Merge into 7.0
Revision history for this message
Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c) wrote :

Hi Nicolas,

Thanks for extracting those modules ! I think it would be better to split this MP in two :

 * One fore sale_sourced_by_line and sale_ownership in this project

 * One for stock_location_ownership in this project I suggest: https://launchpad.net/stock-logistic-warehouse

I know dependencies are together here, but I'm in favor of giving the priority to modules purpose rather than publishing them by dependencies..

Regards,

Joël

review: Needs Fixing (code review + test)

« Back to merge proposal