openerp-community-reviewer team mailing list archive

Thread
Date

Re: lp:~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge into lp:margin-analysis

To: Joël Grand-Guillaume @ camptocamp <joel.grandguillaume@xxxxxxxxxxxxxx>
From: "Maxime Chambreuil \(http://www.savoirfairelinux.com\)" <maxime.chambreuil@xxxxxxxxxxxxxxxxxxxx>
Date: Wed, 11 Dec 2013 15:43:13 -0000
In-reply-to: <20131211122313.17375.98097.launchpad@ackee.canonical.com>
Reply-to: mp+198551@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review and test

The new field "Cost price" is on the Information tab in the product form. It would be better on the Procurements one, with the standard price field.

I have tested the calculation of the cost price based on components cost price and the inventory valuation in the reporting. It is working fine on my side.

Thank you very much Joël.
-- 
https://code.launchpad.net/~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge/+merge/198551
Your team Margin Analysis Core Editors is requested to review the proposed merge of lp:~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge into lp:margin-analysis.