Commit graph

5 commits

Author SHA1 Message Date
Marc Durepos
6fa8068be5 [FIX] delivery_carrier_partner_account - failing collect SO
Fixes a bug where sale orders with a partner_shipping_id not within the
same commercial entity as their partner_id could not have collect
carrier accounts set after being confirmed.

The issue stemmed from the fact that the sale order's recipient_id field
was being set to partner_id, while the created delivery order had
recipient_id set to its partner_id, which is the partner_shipping_id of
the sale order. In other words, the sale order recipient was incorrectly
set to the main partner instead of the shipping address.

This commit adds a test that was previously failing in this scenario. It
also properly sets the recipient_id on the transport selection wizard
and on sale orders themselves, fixing the issue.
2025-07-03 07:18:05 -04:00
Marc Durepos
31892a892a attempt fix to delivery carrier account - selecting collect account raising error 2025-07-03 07:18:05 -04:00
Marc Durepos
4563b8b8b1 Significant rework of delivery_carrier_account.
carrier_id, carrier_account_id and delivery_billing_mode fields are all computed and have a single inverse method that can be overriden by subclasses.
2025-07-03 07:18:05 -04:00
Marc Durepos
a365609f21 [ENH] delivery_carrier_partner_account: add delivery info to sale line
When adding transport charges to a sale order via the "Add Shipping"
button, the line's description now contains the delivery billing mode
and account number when applicable.

Fixes #112
2024-11-21 15:41:45 -05:00
Marc Durepos
9b882ec7f8 delivery_carrier_partner_account: add choose delivery wizard code and tests 2024-11-20 13:13:57 -05:00