← Back to team overview

openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~agilebg/carriers-deliveries/adding_delivery_optional_invoice_line_7 into lp:carriers-deliveries

To: Alex Comba - Agile BG <alex.comba@xxxxxxxxxxx>
From: Sébastien BEAU - http://www.akretion.com <sebastien.beau@xxxxxxxxxxxx>
Date: Fri, 02 May 2014 13:26:14 -0000
In-reply-to: <20140402150223.4527.78579.launchpad@ackee.canonical.com>
Reply-to: mp+213867@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review, no test

Thanks for this module.
Having an additionnal delivery cost is a pain, your module make it optional it's ok for me.
LGTM
-- 
https://code.launchpad.net/~agilebg/carriers-deliveries/adding_delivery_optional_invoice_line_7/+merge/213867
Your team Stock and Logistic Core Editors is subscribed to branch lp:carriers-deliveries.

References

[Merge] lp:~agilebg/carriers-deliveries/adding_delivery_optional_invoice_line_7 into lp:carriers-deliveries
From: Alex Comba - Agile BG, 2014-04-02