-
-
Notifications
You must be signed in to change notification settings - Fork 351
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] delivery_auto_refresh: don't allow to pick a not allowed carrier #469
[FIX] delivery_auto_refresh: don't allow to pick a not allowed carrier #469
Conversation
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.
Shouldn't this be made the same in partner's carrier selection?
Maybe that's a little out of the scope for this module |
774263b
to
cc367cb
Compare
When choosing the sale carrier from the header of the order form we should filter those carriers not available for the given shipping address the in the same way the wizard does it. TT35200
cc367cb
to
27ffc44
Compare
Let's merge it for fixing the bug: /ocabot merge patch Please fw-port it to v14 |
merge failed |
/ocabot merge patch |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 70a9668. Thanks a lot for contributing to OCA. ❤️ |
When choosing the sale carrier from the header of the order form we
should filter those carriers not available for the given shipping
address the in the same way the wizard does it.
cc @Tecnativa TT35200
please review @victoralmau @pedrobaeza