openerp-community-reviewer team mailing list archive

[Nicolas Bessi - Camptocamp <nicolas.bessi@xxxxxxxxxxxxxx>]

Thank for the patch.

Can you please check that bug is still valid in 7.0:
  https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-579462-cpa

Please do not use read when not required it remove readability. 

Please add some doc string it is hard to get into code.


-- 
https://code.launchpad.net/~camptocamp/openerp-product-attributes/7.0-adding-stock_production_lot_custom_attributes-lep/+merge/195950
Your team Product Core Editors is requested to review the proposed merge of lp:~camptocamp/openerp-product-attributes/7.0-adding-stock_production_lot_custom_attributes-lep into lp:openerp-product-attributes.