← Back to team overview

openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~akretion-team/carriers-deliveries/7-split-default-option-state-from-deliv-method-dbl into lp:carriers-deliveries

To: "David BEAL \(ak\)" <david.beal@xxxxxxxxxxxx>
From: "Yannick Vaucher @ Camptocamp" <yannick.vaucher@xxxxxxxxxxxxxx>
Date: Wed, 25 Jun 2014 09:53:48 -0000
In-reply-to: <20140621171622.14137.1896.launchpad@ackee.canonical.com>
Reply-to: mp+224027@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

Ok failed my inline comment

l.102	           <field name="code" readonly="1"/>

You can remove readonly here as you setted it on model
-- 
https://code.launchpad.net/~akretion-team/carriers-deliveries/7-split-default-option-state-from-deliv-method-dbl/+merge/224027
Your team Stock and Logistic Core Editors is subscribed to branch lp:carriers-deliveries.

References

[Merge] lp:~akretion-team/carriers-deliveries/7-split-default-option-state-from-deliv-method-dbl into lp:carriers-deliveries
From: David BEAL (ak), 2014-06-21