Code review comment for lp:~lin-yu/purchase-wkfl/add_product_supplier_info

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

Hi Lin,

Thanks for the corrections, it now LGTM. I mark it as approved, but I just saw one little thing more here :

0 + 'depends': ['product','stock'],#TO REMOVE joomlaconnector for standard

=> You should remove the comment about "joomlconnector", that doesn't make any sense I think

Regards,

Joël

review: Approve (code review, no tests)

« Back to merge proposal