Skip to content
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] users_ldap_mail: Migration to 17.0 #674

Open
wants to merge 18 commits into
base: 17.0
Choose a base branch
from

Conversation

ryanc-me
Copy link

No description provided.

OCA-git-bot pushed a commit that referenced this pull request Nov 10, 2024
* Switch security to be on correct model to fix #674
@cvinh
Copy link

cvinh commented Nov 21, 2024

Hello @ryanc-me are you going to finish the migration ?

alexis-via pushed a commit to akretion/server-auth that referenced this pull request Nov 21, 2024
* Switch security to be on correct model to fix OCA#674
@ryanc-me
Copy link
Author

@cvinh yes, the migration is done - the PR is blocked due to a bug (?) with codecov. we're running this module in production v17 :)

@cvinh
Copy link

cvinh commented Nov 26, 2024

@cvinh yes, the migration is done - the PR is blocked due to a bug (?) with codecov. we're running this module in production v17 :)

Thanks, I can review your PR, can you rebase and put it in Ready for Review ?

dreispt and others added 18 commits November 27, 2024 14:30
ADD defsult and sugegsted values; auto-install
* [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
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/
@ryanc-me ryanc-me marked this pull request as ready for review November 27, 2024 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.