← Back to team overview

openerp-community-reviewer team mailing list archive

Re: lp:~agilebg/account-financial-report/adding_financial_reports_horizontal_7 into lp:account-financial-report

 

Hi Frederic, thank you for your response. The reference to the required field 'reserve_account_id' has been removed in the mean time. This field was removed from OpenERP 7.0, and as a result was not used anymore in this module either. The problem does not occur in my tests. So I think you might want to try and update the module again.

I do see the useless journals tab, it would be good to have this cleaned up. If I am correct, this module does not depend on any common report functionality from OpenERP, so maybe the concept of journal filtering should be severed from the whole module instead of just hiding the tab.

Lorenzo, what do you think?


-- 
https://code.launchpad.net/~agilebg/account-financial-report/adding_financial_reports_horizontal_7/+merge/176235
Your team Account Report Core Editors is subscribed to branch lp:account-financial-report.


Follow ups