← Back to team overview

openerp-community-reviewer team mailing list archive

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