-
-
Notifications
You must be signed in to change notification settings - Fork 493
[18.0][MIG] users_ldap_mail: Migration to version 18.0 #838
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
Conversation
ADD defsult and sugegsted values; auto-install
Fix PEP8
* [ADD] Add README.rst to users_ldap_mail * [MIG] Migrate users_ldap_mail to 10.0 * [FIX] Fixes in code style in module users_ldap_mail Changes in models/users_ldap_mail include: - remove unused imports - indentation of too long lines * [FIX] users_ldap_mail: Fix copyright notice * [FIX] users_ldap_mail: set UTF-8 encoding users_ldap_mail/models/__init__.py should be UTF-8 encoded * [FIX] users_ldap_mail: remove size argument from Char fields * [FIX] users_ldap_mail: return value can be combined in one line * [FIX] Update read me Fixes included: - Make module name human readable - Instead of numbering steps, use # - Update broken link
…inding byte-code strings
Currently translated at 100.0% (5 of 5 strings) Translation: server-auth-16.0/server-auth-16.0-users_ldap_mail Translate-URL: https://translation.odoo-community.org/projects/server-auth-16-0/server-auth-16-0-users_ldap_mail/es/
Currently translated at 100.0% (5 of 5 strings) Translation: server-auth-16.0/server-auth-16.0-users_ldap_mail Translate-URL: https://translation.odoo-community.org/projects/server-auth-16-0/server-auth-16-0-users_ldap_mail/pt_BR/
Currently translated at 100.0% (5 of 5 strings) Translation: server-auth-16.0/server-auth-16.0-users_ldap_mail Translate-URL: https://translation.odoo-community.org/projects/server-auth-16-0/server-auth-16-0-users_ldap_mail/it/
|
/ocabot migration users_ldap_mail |
|
Please check pre-commit error. |
75fd71d to
a43ec96
Compare
pedrobaeza
left a comment
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
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 2190aaf. Thanks a lot for contributing to OCA. ❤️ |
fixup for Migration to 18.0 OCA#838
cc @Tecnativa TT57846
ping @sergio-teruel @pedrobaeza