Review: Approve code review, test
--
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.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to
Review: Approve code review
LGTM, thanks
--
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.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Dear Yannick,
I don't think this MP would break compatibility with
account_financial_report_webkit.
I installed account_financial_report_webkit and account_parralel_currency and I
can use them without problems.
It just doesn't copy the centralized field to the parallel accounts. For that,
a
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
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/account-consolidation/7.0-bug-1296740-elbati into
lp:account-consolidation/7.0.
Requested reviews:
Account Core Editors (account-core-editors)
Related bugs:
Bug #1296740 in Account - Consolidation: 7.0 account_parallel_currency:
Review: Approve code review
LGTM
Regards.
--
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.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
6 matches
Mail list logo
