search

OCA / delivery-carrier

Odoo Carriers And Deliveries Management
https://odoo-community.org/psc-teams/logistics-18
GNU Affero General Public License v3.0
112 stars 351 forks source link

[14.0][IMP] delivery_state: set carrier with no updatable states #844

Closed manuelregidor closed 2 months ago

manuelregidor commented 2 months ago

Before this IMP, the delivery_state was always set to shipping_recorded_in_carrier once the expedition was created. The _update_delivery_state method in stock.picking tries to update the delivery state for those pickings in that state. Because not all the carriers APIs or carrier modules support the state update functionality, this IMP makes it optional by setting the delivery state to no_update instead of shipping_recorded_in_carrier if the new cehckbox track_carrier_state in delivery.carrier model is disabled.

T-6303

manuelregidor commented 2 months ago

@HaraldPanten @ValentinVinagre @luis-ron

HaraldPanten commented 2 months ago

@pedrobaeza @chienandalu Do you mind merging this? THX

OCA-git-bot commented 2 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

pedrobaeza commented 2 months ago

/ocabot merge patch

OCA-git-bot commented 2 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 14.0-ocabot-merge-pr-844-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 2 months ago

Congratulations, your PR was merged at 6d84d56090738cb43d3e891ec5ff2aab87502962. Thanks a lot for contributing to OCA. ❤️