← Back to team overview

banking-addons-drivers team mailing list archive

Re: Banking addons guidelines for 8.0 - a proposal

 

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 

----- Mail original -----



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: 

<blockquote>



Dear Banking addons friends, 


Le 10/10/2014 18:15, Bidoul, Stéphane a écrit : 

<blockquote>

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 


</blockquote>


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


</blockquote>


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