← Back to team overview

openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~camptocamp/margin-analysis/7.0-product_get_cost_field-standard_price-1308584 into lp:margin-analysis

To: mp+216124@xxxxxxxxxxxxxxxxxx
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Wed, 16 Apr 2014 16:42:26 -0000
In-reply-to: <20140416143921.29408.15328.launchpad@ackee.canonical.com>
Reply-to: mp+216124@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review

LGTM.

I avoid always I can to make a "replace" operation, making invisible the field instead.

Regards.
-- 
https://code.launchpad.net/~camptocamp/margin-analysis/7.0-product_get_cost_field-standard_price-1308584/+merge/216124
Your team Margin Analysis Core Editors is subscribed to branch lp:margin-analysis.

References

[Merge] lp:~camptocamp/margin-analysis/7.0-product_get_cost_field-standard_price-1308584 into lp:margin-analysis
From: Guewen Baconnier @ Camptocamp, 2014-04-16