← Back to team overview

account-payment-team team mailing list archive

Thread
Date

Re: [Merge] lp:~pedro.baeza/account-payment/6.1-set_done_optimization into lp:account-payment

To: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
From: Joël Grand-Guillaume @ camptocamp <joel.grandguillaume@xxxxxxxxxxxxxx>
Date: Thu, 17 Oct 2013 12:09:50 -0000
In-reply-to: <20130830170041.6094.66092.launchpad@ackee.canonical.com>
Reply-to: mp+183234@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review, no tests

Hi,

Thank you for the contribution ! I would have explicitly written context=context (or ctx) in lines : 14, 79, 108, 125, 144, 153.

Otherwise good !

Thanks,


-- 
https://code.launchpad.net/~pedro.baeza/account-payment/6.1-set_done_optimization/+merge/183234
Your team Account Payment is subscribed to branch lp:account-payment.

References

[Merge] lp:~pedro.baeza/account-payment/6.1-set_done_optimization into lp:account-payment
From: Pedro Manuel Baeza, 2013-08-30