openerp-community-reviewer team mailing list archive

[Nicolas Bessi - Camptocamp <nicolas.bessi@xxxxxxxxxxxxxx>]

Review: Approve no test, code review

Hello, 

I agree with Alexandre and Guewen. 

I generally do such tests in opposite situation,
when a public function is waiting for one id list,in and where nested form can do the call to a button, on change, webclient is not always consistent... 

I fixed an indentation and remove unused _columns 

LGTM
-- 
https://code.launchpad.net/~camptocamp/stock-logistic-flows/7.0-add_stock_picking_priority-afe/+merge/197206
Your team Stock and Logistic Core Editors is subscribed to branch lp:stock-logistic-flows/7.0.