-
-
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
[17.0][MIG] delivery_carrier_info #779
[17.0][MIG] delivery_carrier_info #779
Conversation
Currently translated at 42.8% (3 of 7 strings) Translation: delivery-carrier-14.0/delivery-carrier-14.0-delivery_carrier_info Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-14-0/delivery-carrier-14-0-delivery_carrier_info/fr/
Currently translated at 100.0% (3 of 3 strings) Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_carrier_info Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_carrier_info/es/
c0aa851
to
f257c9f
Compare
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.
LGTM! Code and functional review 👍🏽
/ocabot migration delivery_carrier_info |
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.
Code review.
I know this has been discussed but I'm wondering if this deserves a base module to add a group in forms and then add 'code' and so on...
This PR has the |
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 0a79fc7. Thanks a lot for contributing to OCA. ❤️ |
Standard Migration
@ForgeFlow