-
-
Notifications
You must be signed in to change notification settings - Fork 435
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] auth_admin_passkey: Migration to 18.0 #710
[18.0] [MIG] auth_admin_passkey: Migration to 18.0 #710
Conversation
…o login with any login and his password.
…ion if user_id == SUPERUSER_ID. (Courtesy of Stefan Rijnhart).
…s the same password as the admin user, sending a mail to admin user.
… installation of modules 'auth_admin_passkey' and 'auth_crypt'. (Courtesy of Stefan Rijnhart).
…xxx'] [REF] res_users.py : same call to self.pool in other res_users class.
In order to get visibility on https://www.odoo.com/apps the OCA board has decided to add the OCA as author of all the addons maintained as part of the association.
return subject, "<pre>{}</pre>".format(body) | ||
return subject, f"<pre>{body}</pre>"` | ||
|
||
looks lighter. |
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.
sorry. something has been wrong in the github proposal !
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.
@legalsylvain I fixed this issue. Now I have the following issue:
AttributeError: 'dict' object has no attribute 'encode' on line:
password = hashlib.sha512(password.encode()).hexdigest()
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.
it looks quite unrelated, don't you think ?
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.
@legalsylvain Yes your solution fixed the previous problem. It's not related to this one.
@moncefarajdal hello, need to migrate this module and found out this Closed PR. |
No description provided.