← Back to team overview

openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~sistemas-adhoc/account-financial-tools/currency_update into lp:account-financial-tools

To: mp+203755@xxxxxxxxxxxxxxxxxx
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Sun, 02 Feb 2014 11:04:57 -0000
In-reply-to: <20140129142123.21482.98375.launchpad@ackee.canonical.com>
Reply-to: mp+203755@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

Good improvement, thanks, but instead of print statements, could you please raise orm.except_orm exception?

Regards.
-- 
https://code.launchpad.net/~sistemas-adhoc/account-financial-tools/currency_update/+merge/203755
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-financial-tools.

References

[Merge] lp:~sistemas-adhoc/account-financial-tools/currency_update into lp:account-financial-tools
From: Juan José Scarafía, 2014-01-29