-
Notifications
You must be signed in to change notification settings - Fork 106
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
[12.0][IMP] shopinvader_delivery_carrier change args order #615
[12.0][IMP] shopinvader_delivery_carrier change args order #615
Conversation
/ocabot merge minor |
This PR looks fantastic, let's merge it! |
@sebastienbeau your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-615-by-sebastienbeau-bump-minor. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
This PR has the |
/ocabot merge minor |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 4108512. Thanks a lot for contributing to shopinvader. ❤️ |
Change args order to be consistent with ./delivery_carrier.py/_perpare_carrier
https://github.com/shopinvader/odoo-shopinvader/blob/12.0/shopinvader_delivery_carrier/services/delivery_carrier.py#L113