banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #01941
Re: [Merge] lp:~akretion-team/banking-addons/bank-statement-reconcile-70-multi-statements into lp:banking-addons/bank-statement-reconcile-7.0
Review: Needs Information
Hi Florian,
I've implemented the multi statement support in an addon to import the bank statements from a standard CODA file (belgium standard).
The implementation use the draft proposal unchanged. ( http://bazaar.launchpad.net/~acsone-openerp/banking-addons/bank-statement-reconcile-70-multi-statements/revision/147)
As expected, the changes to the parser were minor and seem to take advantage of existing methods.
http://bazaar.launchpad.net/~acsone-openerp/+junk/coda-completion/revision/14
http://bazaar.launchpad.net/~acsone-openerp/+junk/coda-completion/view/head:/account_statement_coda_import/parser/coda_file_parser.py
What do you think of this approach?
Regards,
lmi
--
https://code.launchpad.net/~akretion-team/banking-addons/bank-statement-reconcile-70-multi-statements/+merge/209863
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons/bank-statement-reconcile-7.0.
References