Just quick things I noticed:
- 'auto_install'should be used instead of 'active' in v7.
- Are we creating a module for one field? May be its better to add this into 
parent module (lp:openerp-product-attributes) if product model is used there.

Just an opinion,
Thanks.

-- 
https://code.launchpad.net/~agilebg/openerp-product-attributes/adding_product_category_code_7/+merge/176734
Your team OpenERP Community is subscribed to branch 
lp:openerp-product-attributes.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp

Reply via email to