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. -- Mailing list: https://launchpad.net/~banking-addons-team Post to : [email protected] Unsubscribe : https://launchpad.net/~banking-addons-team More help : https://help.launchpad.net/ListHelp
