openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~camptocamp/account-financial-tools/7.0-batch-validate-log-more-lep into lp:account-financial-tools

To: Leonardo Pistone - camptocamp <leonardo.pistone@xxxxxxxxxxxxxx>
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Wed, 26 Feb 2014 23:04:05 -0000
In-reply-to: <20140226162529.29105.51612.launchpad@ackee.canonical.com>
Reply-to: mp+208420@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review

Hi, Leonardo, I think it's better a more neutral language on logging messages. Could you put something like "{0} jobs for posting moves will be created"?

Regards.
-- 
https://code.launchpad.net/~camptocamp/account-financial-tools/7.0-batch-validate-log-more-lep/+merge/208420
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-financial-tools.

References

[Merge] lp:~camptocamp/account-financial-tools/7.0-batch-validate-log-more-lep into lp:account-financial-tools
From: Leonardo Pistone - camptocamp, 2014-02-26