[13.0] Add stock_picking_type_shipping_policy - alpha #18
Conversation
grindtildeath
changed the title
grindtildeath
changed the title
grindtildeath
force-pushed
the
13.0-add-stock_picking_type_shipping_policy
branch
from
320b342 to
f6139f7
Compare
simahawk
changed the title
|
|
||
| shipping_policy = fields.Selection( | ||
| SHIPPING_POLICY_SELECTION, | ||
| default=lambda r: r._default_shipping_policy(), |
There was a problem hiding this comment.
I think you don't need to split selection options to a var. You could delegate them to a method instead.
You don't need the default method but if you move options to a method then it's better if you keep it.
| @@ -0,0 +1,2 @@ | |||
| This module adds a Shipping Policy field on Operations Types in order to force | |||
| a specific Shipping Policy on Pickings according to their types. | |||
There was a problem hiding this comment.
maybe worth to mention the use case?
There was a problem hiding this comment.
Could be:
This is especially useful if you use a pick-pack-ship setup with the release of operation (stock_available_to_promise_release) module along side with the stock_routing_operation that may split operations by zone of the warehouse. In that case, you want to be sure the pack operations will wait all different picks to be processed before releasing the availability of the pack operation.
| @@ -0,0 +1,2 @@ | |||
| This module adds a Shipping Policy field on Operations Types in order to force | |||
| a specific Shipping Policy on Pickings according to their types. | |||
There was a problem hiding this comment.
Could be:
This is especially useful if you use a pick-pack-ship setup with the release of operation (stock_available_to_promise_release) module along side with the stock_routing_operation that may split operations by zone of the warehouse. In that case, you want to be sure the pack operations will wait all different picks to be processed before releasing the availability of the pack operation.
|
/ocabot merge |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at e66f975. Thanks a lot for contributing to OCA. ❤️ |
backend: run tests as a stock user
This module adds a Shipping Policy field on Operations Types in order to force
a specific Shipping Policy on Pickings according to their types.