banking-addons-team team mailing list archive

Thread
Date

Re: lp:~akretion-team/banking-addons/bank-statement-reconcile-70-improve-import into lp:banking-addons/bank-statement-reconcile-7.0

To: Sébastien BEAU - http://www.akretion.com <sebastien.beau@xxxxxxxxxxxx>
From: Joël Grand-Guillaume @ camptocamp <joel.grandguillaume@xxxxxxxxxxxxxx>
Date: Wed, 11 Dec 2013 14:11:33 -0000
In-reply-to: <20131204173645.13946.42119.launchpad@ackee.canonical.com>
Reply-to: mp+197759@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review, no tests

Hi Sebastien,

Thanks for the contribs ! Just a little remark on lines 45 to 57, you should review and adpat your comment.

Otherwise LGTM.

Regards,

Joël
-- 
https://code.launchpad.net/~akretion-team/banking-addons/bank-statement-reconcile-70-improve-import/+merge/197759
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons/bank-statement-reconcile-7.0.

References

lp:~akretion-team/banking-addons/bank-statement-reconcile-70-improve-import into lp:banking-addons/bank-statement-reconcile-7.0
From: Sébastien BEAU - http : //www . akretion . com, 2013-12-04