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

Revision history for this message
Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c) wrote :

Hi Xavier,

I change everything according to your comments. I hope this is fine for you now.

Summary:

- Change the stocks wizards (both with amount_unit)
- Remove commented code (hope I don't miss some...)
- Change <> for !=

Note, I don't find the ".has_key" you talked about. So, in case of trouble with this, feel free to rewrite that point for the merge.

Thanks in advance for merging this !

Regards,

Joël

« Back to merge proposal