openerp-community-reviewer team mailing list archive

Thread
Date

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

To: Stéphane Bidoul (Acsone) <stephane.bidoul@xxxxxxxxx>
From: Lorenzo Battistini - Agile BG <lorenzo.battistini@xxxxxxxxxxx>
Date: Tue, 22 Apr 2014 07:45:33 -0000
In-reply-to: <20140418130420.28894.17308.launchpad@ackee.canonical.com>
Reply-to: mp+216442@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Information

Hello Stéphane,

why not using '_constraints' member instead of overriding create and write methods?

Is there any technical reason to avoid YAML tests?

Thanks!
-- 
https://code.launchpad.net/~acsone-openerp/account-financial-tools/account_partner_required-sbi/+merge/216442
Your team Account Core Editors is requested to review the proposed merge of lp:~acsone-openerp/account-financial-tools/account_partner_required-sbi into lp:account-financial-tools.

References

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