-
-
Notifications
You must be signed in to change notification settings - Fork 179
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
[18.0][MIG] base_iso3166: Migration to 18.0 #220
Conversation
Thanks for the contribution. Please preserve commit history following technical method explained in https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0. If the jump is between several versions, you have to modify the source branch in the main command to accommodate it to this circumstance. |
=========================== ISO 3166 adaptation to Odoo =========================== This module adds some data specified by ISO 3166, specifically: * ISO 3166-1 alpha-3 (3-char) codes in countries * ISO 3166-1 numeric codes in countries * ISO 3166-3 formerly used codes in countries Check https://en.wikipedia.org/wiki/ISO_3166 for more info. Installation ============ This module requires the Python library pycountry >= 0.19 installed in the system. Known issues / Roadmap ====================== * Include 3166-2 country subdivision codes
Fix readme Fix manifest version Test coverage Delete author
With newer library versions, some countries are not found, so we have to protect that situation.
dc7c727
to
dbdc8ed
Compare
Thanks a lot for your feedback, I've just refactored my branch following the migration method. |
/ocabot migration base_iso3166 |
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 and seeing that the fields are filled correctly 👍
ping @pedrobaeza |
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.
/ocabot merge nobump
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 2383e47. Thanks a lot for contributing to OCA. ❤️ |
Hi everyone,
This is my first contribution, do not hesitate to tell me if i am doing wrong and i will try to update
Thanks