Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] [16.0] delivery_auto_refresh: Allow override carrier from wizard when changing partner #914

Conversation

Shide
Copy link
Contributor

@Shide Shide commented Dec 5, 2024

Always overrides the carrier when partner or delivery partner changes

closes #873

MT-6943 @moduon @rafaelbn @Gelojr @fcvalgar @jbaudoux @simahawk please review if you want :)

@Shide Shide marked this pull request as draft December 5, 2024 11:35
@Shide Shide marked this pull request as ready for review December 5, 2024 11:40
@Shide
Copy link
Contributor Author

Shide commented Dec 5, 2024

@jbaudoux why you don't want to override carrier when changing partner? I'm missing something, or you do this to cover some case?

@rafaelbn rafaelbn added this to the 16.0 milestone Dec 5, 2024
Copy link

@Gelojr Gelojr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Shide Shide force-pushed the 16.0-auto_update_delivery_carrier-delivery_auto_refresh branch from 5d21a05 to bdff1b3 Compare December 10, 2024 07:05
@Shide Shide requested review from jbaudoux and simahawk December 10, 2024 07:07
@Shide
Copy link
Contributor Author

Shide commented Dec 10, 2024

@jbaudoux done as I think you prefer it.

Note that sale_order_carrier_auto_assign on v17 has not been migrated following your Migration notes (function has not been renamed) and may need extra work when this change goes to v17 on a migration (I don't think I migrate this module because we are skipping v17 version)

@jbaudoux
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-914-by-jbaudoux-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 05683c9 into OCA:16.0 Dec 10, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@Shide Shide deleted the 16.0-auto_update_delivery_carrier-delivery_auto_refresh branch December 10, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[16] delivery_auto_refresh doesn't update if customer is changed
7 participants