← Back to team overview

banking-addons-team team mailing list archive

Thread
Date

Re: lp:~camptocamp/banking-addons/bank-statement-reconcile-70-restore_default_value into lp:banking-addons/bank-statement-reconcile-70

To: mp+154650@xxxxxxxxxxxxxxxxxx
From: "Guewen Baconnier @ Camptocamp" <guewen.baconnier@xxxxxxxxxxxxxx>
Date: Tue, 02 Apr 2013 11:31:19 -0000
In-reply-to: <20130325103536.27651.29157.launchpad@ackee.canonical.com>
Reply-to: mp+154650@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review, no test

invisible="True" at l.36 is weird, shouldn't it be a boolean instead of a str?

Before l.21 you should ensure that context is a dict.
-- 
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-70-restore_default_value/+merge/154650
Your team Banking Addons Team is subscribed to branch lp:banking-addons/bank-statement-reconcile-70.

References

lp:~camptocamp/banking-addons/bank-statement-reconcile-70-restore_default_value into lp:banking-addons/bank-statement-reconcile-70
From: Nicolas Bessi - Camptocamp, 2013-03-25