← 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

 

Hello Pedro,

account_payment_extension and account_banking_payment(_export) are indeed
incompatible and have overlapping functionalities.

I don't know if it is possible to resolve them short of refactoring them
both in a suite of smaller modules. Ideas in that area are most welcome.

-sbi


On Fri, Aug 16, 2013 at 7:12 PM, Pedro Manuel Baeza
<pedro.baeza@xxxxxxxxx>wrote:

> 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
> You are subscribed to branch lp:banking-addons.
>

-- 
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.


References