openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~acsone-openerp/account-financial-tools/allow_date_fy-sbi into lp:account-financial-tools

To: Stéphane Bidoul (Acsone) <stephane.bidoul@xxxxxxxxx>
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Tue, 01 Apr 2014 14:01:58 -0000
In-reply-to: <20140401125145.1728.45636.launchpad@ackee.canonical.com>
Reply-to: mp+213644@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review

Hi, Stéphane,

Thanks for the MP. I would name allow_date_fy variable as check_date_fy to better reflect the current behaviour, but that's not a blocking issue.

Regards.
-- 
https://code.launchpad.net/~acsone-openerp/account-financial-tools/allow_date_fy-sbi/+merge/213644
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-financial-tools.

Follow ups

Re: [Merge] lp:~acsone-openerp/account-financial-tools/allow_date_fy-sbi into lp:account-financial-tools
From: Acsone, 2014-04-01

References

[Merge] lp:~acsone-openerp/account-financial-tools/allow_date_fy-sbi into lp:account-financial-tools
From: Acsone, 2014-04-01