Code review comment for lp:~jgrandguillaume-c2c/openobject-addons/multi-company-cost-price

Revision history for this message
Raphaƫl Valyi - http://www.akretion.com (rvalyi) wrote :

Hello, one thing that I can't understand is why you add a field in product and views called "standard_price_usd" by default?
Is that a typo?
If not, I think we shouldn't favor some country price by default, especially if there is just one like that. I agree that often you would need such field, but then I think you should rather add it at integration time, else is seems to me a bit arbitrary.
If that's a typo, please make sure you don't have such typo in your merge proposal.

« Back to merge proposal