[18.0] [ADD] stock_dynamic_routing_delivery #2109
Conversation
|
Hi @jbaudoux, |
ivantodorovich
force-pushed
the
stock-dynamic-routing-delivery
branch
from
7471c5e to
6061a39
Compare
| and routing_rule.propagate_carrier | ||
| and not vals.get("carrier_id") | ||
| ): | ||
| vals["carrier_id"] = self.group_id.sale_id.carrier_id.id |
There was a problem hiding this comment.
I don't know how odoo is doing to propagate the carrier but shouldn't come from the sale. You better take it from the procurement group. There is an oca module adding it there.
There was a problem hiding this comment.
I tried to copy what Odoo does for their own propagate_carrier:
In STD there's no carrier_id in the procurement group, we'd need an extra dependency (delivery_procurement_group_carrier) but it makes no sense IMO
There was a problem hiding this comment.
Issue is that we do a carrier selection on the delivery. So we can't rely on the SO carrier.
I fixed the odoo rule propagation here #2110
There was a problem hiding this comment.
I added also a glue module: stock_dynamic_routing_delivery_procurement_group_carrier.
Does it make sense to you?
ivantodorovich
force-pushed
the
stock-dynamic-routing-delivery
branch
from
6061a39 to
04f0ba1
Compare
ivantodorovich
force-pushed
the
stock-dynamic-routing-delivery
branch
from
04f0ba1 to
3b5d7ce
Compare
3 participants
No description provided.