banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #00822
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