banking-addons-team team mailing list archive

Thread
Date

Re: lp:~akretion-team/banking-addons/bank-statement-reconcile-7.0-one-move into lp:banking-addons/bank-statement-reconcile-7.0

To: Sébastien BEAU - http://www.akretion.com <sebastien.beau@xxxxxxxxxxxx>
From: Sébastien BEAU - http://www.akretion.com <sebastien.beau@xxxxxxxxxxxx>
Date: Mon, 13 Jan 2014 10:11:38 -0000
In-reply-to: <20131204175532.32518.98756.launchpad@ackee.canonical.com>
Reply-to: mp+197769@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Hi Guewen,
Florian will fix the minor change.
Regarding the line 190, I re-use the code from here http://bazaar.launchpad.net/~openerp/openobject-addons/7.0/view/head:/account/account_bank_statement.py#L320. I implement exactly the same behaviour in order to be consistent, we can change it be in this case I think we should also ask a refactor in account module.
Note I didn't test a multi-currency case with my module as I don't have one. If Frederic can test one it can be a good idea.

-- 
https://code.launchpad.net/~akretion-team/banking-addons/bank-statement-reconcile-7.0-one-move/+merge/197769
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons/bank-statement-reconcile-7.0.