banking-addons-team team mailing list archive

Thread
Date

Re: [Merge] lp:~akretion-team/banking-addons/bank-statement-reconcile-70-account-not-mandatory into lp:banking-addons/bank-statement-reconcile-7.0

To: Sébastien BEAU - http://www.akretion.com <sebastien.beau@xxxxxxxxxxxx>
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Thu, 10 Apr 2014 08:28:08 -0000
In-reply-to: <20131204170826.10051.54641.launchpad@ackee.canonical.com>
Reply-to: mp+197755@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review

Hi, Florian, I still see two issues:

- On l.23, length is more than 79 cols.
- On l.44-48, there is a conflict with the sources you have merged. Please resolve it.

Regards.
-- 
https://code.launchpad.net/~akretion-team/banking-addons/bank-statement-reconcile-70-account-not-mandatory/+merge/197755
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-account-not-mandatory into lp:banking-addons/bank-statement-reconcile-7.0
From: Sébastien BEAU - http : //www . akretion . com, 2013-12-04