← Back to team overview

banking-addons-drivers team mailing list archive

Re: Banking addons guidelines for 8.0 - a proposal

 

I fully agree with the Stephane's proposition. It brings a cleaner and
clearer view of the modules.

So +1 for this!

Alexis, have you already a WIP branch for the paypal import?

thanks

2014-11-10 20:43 GMT+01:00 Maxime Chambreuil <
maxime.chambreuil@xxxxxxxxxxxxxxxxxxxx>:

> Hello,
>
> I have few experience with the subject, but
>
>    - if I need to install to 2 repo to import a statement from a bank and
>    do the reconciliation, then so be it. Anyway, we plan to have the
>    bank-specific module in our localization...
>    - I would keep oca/account-payment to host customizations to
>    account.voucher (like supplier_invoice_number). Someone may still need them
>    without any integration with its bank.
>
> My 2 cents.
> --
> Maxime Chambreuil
> +1 (514) 276-5468 #126
>
> ------------------------------
> Yeah, sorry for not answer here. I agree with the proposal and in the PR I
> argument my position. Quote from there:
>
> *"It's more a cleaning strategy than an area split decission. Having this
> new repo for import, we start from scratch and all modules present in it
> are with the new components, and we can even backporting them to v7, but
> having in the same repo, user will be confused about what modules to use."*
>
> Regards.
>
> 2014-10-30 18:31 GMT+01:00 Bidoul, Stéphane <stephane.bidoul@xxxxxxxxx>:
>
>> Hi Alexis,
>>
>> Thanks for your input.
>>
>> Therp has started working on the backport, reusing your initial work:
>> https://github.com/OCA/banking/pull/53
>>
>> In the comments of that PR we have Pedro agreeing to my proposal and
>> Ronald Portier who is pointing out it would require users of banking addons
>> to checkout 3 repositories.
>>
>> Cheers,
>>
>> -sbi
>>
>> On Thu, Oct 30, 2014 at 6:23 PM, Alexis de Lattre <alexis@xxxxxxxxxx>
>> wrote:
>>
>>>  Dear Banking addons friends,
>>>
>>> Le 10/10/2014 18:15, Bidoul, Stéphane a écrit :
>>>
>>> If the community agrees on these conclusions, I would suggest the
>>> following as guidelines for managing the ports to 8.0:
>>>
>>>    - deprecate OCA/account-payment (ie no 8.0 branch)
>>>    - OCA/banking becomes the repository for payment interfaces
>>>       - possibly renamed to OCA/bank-payment
>>>       - only the payment part gets ported to the 8.0 branch (in
>>>       progress [3] [4])
>>>    - OCA/bank-statement-reconcile becomes the repository for statement
>>>    completion and reconciliation
>>>       - the statement import features in it are not ported to the 8.0
>>>       branch
>>>    - a new repository named OCA/bank-statement-import is created
>>>       - it hosts a backport of odoo master
>>>       branch account_bank_statement_import
>>>       - as well as parsers and additional import features modules
>>>       inspired by OCA/banking and OCA/bank-statement-reconcile 7.0
>>>
>>> Looking forward to reading your feedback,
>>>
>>>
>>> I fully agree with your proposal Stéphane.
>>>
>>> Support for French CFONB via the account_bank_statement_import is
>>> already available in the French localisation for v8 :
>>> http://bazaar.launchpad.net/~openerp-fr-core-editors/openerp-french-localization/8.0/files/head:/account_bank_statement_import_fr_cfonb/
>>>
>>> I have been investigating some issues with the OFX format for start/end
>>> balance:
>>> https://github.com/odoo/odoo/issues/3003
>>> But, apart from that, the OFX import is working well and I see a lot of
>>> French banks that propose OFX files.
>>>
>>> I am about to write a Paypal bank statement import (CSV format). That
>>> would be an additionnal module for the future branch
>>> OCA/bank-statement-import.
>>>
>>> Alexis
>>>
>>> --
>>> Mailing list: https://launchpad.net/~banking-addons-drivers
>>> Post to     : banking-addons-drivers@xxxxxxxxxxxxxxxxxxx
>>> Unsubscribe : https://launchpad.net/~banking-addons-drivers
>>> More help   : https://help.launchpad.net/ListHelp
>>>
>>>
>>
>> --
>> Mailing list: https://launchpad.net/~banking-addons-drivers
>> Post to     : banking-addons-drivers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~banking-addons-drivers
>> More help   : https://help.launchpad.net/ListHelp
>>
>>
>
> --
> Mailing list: https://launchpad.net/~banking-addons-drivers
> Post to     : banking-addons-drivers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~banking-addons-drivers
> More help   : https://help.launchpad.net/ListHelp
>
>
> --
> Mailing list: https://launchpad.net/~banking-addons-drivers
> Post to     : banking-addons-drivers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~banking-addons-drivers
> More help   : https://help.launchpad.net/ListHelp
>
>

Follow ups

References