Review: Needs Fixing code review, no test

Thank you Lorenzo, Alex.

- Translations, if they are old and/or incorrect, IMO we can remove them for 
now. You could leave whatever works for you (i.e. Italian) and remove 
everything else

- Defaults: lambda is not needed anymore

- I see some long, multi-line comments in the code. Maybe some could be moved 
in some more prominent place, like docstrings?

L
-- 
https://code.launchpad.net/~agilebg/openerp-product-attributes/adding_product_pack_7/+merge/195786
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