← Back to team overview

openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~camptocamp/stock-logistic-flows/7.0-stock_picking_priority-selection-extendable into lp:stock-logistic-flows

To: mp+208747@xxxxxxxxxxxxxxxxxx
From: Leonardo Pistone - camptocamp <leonardo.pistone@xxxxxxxxxxxxxx>
Date: Tue, 11 Mar 2014 09:18:38 -0000
In-reply-to: <20140228085034.24070.31140.launchpad@ackee.canonical.com>
Reply-to: mp+208747@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review

LGTM Guewen.

small remark: would it make sense to add the docstring to the other 2 __selection_priority methods?
-- 
https://code.launchpad.net/~camptocamp/stock-logistic-flows/7.0-stock_picking_priority-selection-extendable/+merge/208747
Your team Stock and Logistic Core Editors is subscribed to branch lp:stock-logistic-flows.

Follow ups

Re: [Merge] lp:~camptocamp/stock-logistic-flows/7.0-stock_picking_priority-selection-extendable into lp:stock-logistic-flows
From: Guewen Baconnier @ Camptocamp, 2014-03-12

References

[Merge] lp:~camptocamp/stock-logistic-flows/7.0-stock_picking_priority-selection-extendable into lp:stock-logistic-flows
From: Guewen Baconnier @ Camptocamp, 2014-02-28