openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~camptocamp/account-financial-report/add_aged_partner_ledger-nbi into lp:account-financial-report

To: mp+211040@xxxxxxxxxxxxxxxxxx
From: Stéphane Bidoul (Acsone) <stephane.bidoul@xxxxxxxxx>
Date: Sun, 13 Apr 2014 13:54:48 -0000
In-reply-to: <20140317121326.10185.63277.launchpad@ackee.canonical.com>
Reply-to: mp+211040@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve test

Report works fine provided the clearance date is set correctly. 

My only remark is a usability issue as the most common use case (aged balance as of current date) requires fiddling with the date filters. As another MP resolving the usability issue is on its way, I approve this one.

Thanks!
-- 
https://code.launchpad.net/~camptocamp/account-financial-report/add_aged_partner_ledger-nbi/+merge/211040
Your team Account Report Core Editors is subscribed to branch lp:account-financial-report.

References

[Merge] lp:~camptocamp/account-financial-report/add_aged_partner_ledger-nbi into lp:account-financial-report
From: Nicolas Bessi - Camptocamp, 2014-03-17