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

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Pedro, I agree that it would have been better if the question was answered before the merge.
Now, I'm sure there was an excellent reason why the existing field couldn't be used, even if I can't remember what it is; Joël thought about this module and I remember asking him the exact same question, so there was a reason. Joël if you can answer...

I suggest that we propose a merge with the .pot and maybe improve the description of the module so it shows what it brings compared to the existing field.

« Back to merge proposal