← Back to team overview

openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~acsone-openerp/account-financial-report/7.0-bug-1281579-sbi2 into lp:account-financial-report

To: Stéphane Bidoul (Acsone) <stephane.bidoul@xxxxxxxxx>
From: Leonardo Pistone - camptocamp <leonardo.pistone@xxxxxxxxxxxxxx>
Date: Mon, 07 Jul 2014 10:17:59 -0000
In-reply-to: <20140704092222.22254.6037.launchpad@ackee.canonical.com>
Reply-to: mp+225620@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

Stéphane,

Maybe you could refactor a bit with:

cur_year = time.strftime('%Y')
and then use it later?

being that a small change, I approve anyway
-- 
https://code.launchpad.net/~acsone-openerp/account-financial-report/7.0-bug-1281579-sbi2/+merge/225620
Your team Account Report Core Editors is subscribed to branch lp:account-financial-report.

References

[Merge] lp:~acsone-openerp/account-financial-report/7.0-bug-1281579-sbi2 into lp:account-financial-report
From: Acsone, 2014-07-04