banking-addons-team team mailing list archive

[Joël Grand-Guillaume @ camptocamp <joel.grandguillaume@xxxxxxxxxxxxxx>]

Review: Approve code review, no tests

Hi,


Thanks for the work. Regarding Sebastien's remark about monking patching, I think we can avoid it here. The only purpose of this module IS to change the behavior of the numbering by assuring that regardless what OpenERP or what ever other module change this, the logic introduced here remain. The goal to to be able to rely on what is in the descrition.

So theoretically, I agree with Seb, but the purpose here is to ensure this behavior and not another.

So I mark this one as Approved.

Regards,

Joël
-- 
https://code.launchpad.net/~camptocamp/banking-addons/7.0-bank-statement-reconcile-account_invoice_reference/+merge/212099
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons/bank-statement-reconcile-7.0.