-
-
Notifications
You must be signed in to change notification settings - Fork 366
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
[FIX] [16.0] delivery_auto_refresh: Allow override carrier from wizard when changing partner #914
Conversation
@jbaudoux why you don't want to override carrier when changing partner? I'm missing something, or you do this to cover some case? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…hanges on the order
5d21a05
to
bdff1b3
Compare
@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) |
/ocabot merge patch |
On my way to merge this fine PR! |
Congratulations, your PR was merged at c266410. Thanks a lot for contributing to OCA. ❤️ |
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 :)