← Back to team overview

banking-addons-team team mailing list archive

Re: lp:~therp-nl/banking-addons/ba70-payment_order_reconciliation_line_confusion into lp:banking-addons

 

Review: Approve code review, no test

LGTM.

Now, please allow me a little off topic refering this module: in Spain we have this type of operations depending on account-payment-extension, with a module called l10n_es_payment_order. For the 7.0 spanish localization, I'm personally putting a lot of effort in going with the most standard way of doing things. This includes banking payments exports and statements import, but OpenERP is also working on a new system that will include on base SEPA import and export. What do you think? We must wait in 7 backporting current modules to see this features and inherit from them in v8 or go on with the community way and change the core now?

Regards.
-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba70-payment_order_reconciliation_line_confusion/+merge/180590
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~therp-nl/banking-addons/ba70-payment_order_reconciliation_line_confusion into lp:banking-addons.


Follow ups

References