← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~agilebg/account-consolidation/7.0-bug-1296740-elbati into lp:account-consolidation/7.0

 

Review: Needs Fixing

centralized is defined in account-financial-report/account_financial_report_webkit/account.py

It seams there is no dependancy on it and I think no dependancy should be there.
But simply removing it will break compatibility with account_financial_report_webkit

However, a link module by extracting centralized can be added to ensure account_financial_report_webkit and account_parralel_currency are still compatible.

Can you extract centralized copying in a new module in auto-install when account_financial_report_webkit and account_parralel_currency are installed?
-- 
https://code.launchpad.net/~agilebg/account-consolidation/7.0-bug-1296740-elbati/+merge/212487
Your team Account Core Editors is subscribed to branch lp:account-consolidation/7.0.


References