openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #06662
Re: [Merge] lp:~camptocamp/account-invoicing/account-analytic-required-vre into lp:account-invoicing
Review: Needs Fixing code review
#60 sale.invoice.line?
#104 _description is not necessary here
#180ff you should remove translation files that don't actually contain translations
And per default, the field is only visible for analytic.group_analytic_accounting. I think this addon should remove this group from the view definition.
--
https://code.launchpad.net/~camptocamp/account-invoicing/account-analytic-required-vre/+merge/218974
Your team Account Core Editors is subscribed to branch lp:account-invoicing.
References