← Back to team overview

openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~camptocamp/account-financial-tools/6.1-fix-1276998 into lp:account-financial-tools/6.1

To: mp+205121@xxxxxxxxxxxxxxxxxx
From: Alexandre Fayolle - camptocamp <alexandre.fayolle@xxxxxxxxxxxxxx>
Date: Thu, 06 Feb 2014 12:27:54 -0000
In-reply-to: <20140206105921.441.53092.launchpad@ackee.canonical.com>
Reply-to: mp+205121@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review, no test

line 55 could use self.logger.error(error_msg) (ie. change the logging level and use a more informative message)
-- 
https://code.launchpad.net/~camptocamp/account-financial-tools/6.1-fix-1276998/+merge/205121
Your team Account Core Editors is subscribed to branch lp:account-financial-tools/6.1.

References

[Merge] lp:~camptocamp/account-financial-tools/6.1-fix-1276998 into lp:account-financial-tools/6.1
From: Yannick Vaucher @ Camptocamp, 2014-02-06