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

[t154502][18.0][MIG] auth_admin_passkey: Migration to 18.0 #523

Merged

Conversation

BT-dlagin
Copy link

No description provided.

legalsylvain and others added 30 commits December 20, 2024 11:03
…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.
[ADD] Test #1 to OCA#5 to avoid regression;
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.
alexis-via and others added 21 commits December 20, 2024 11:03
Currently translated at 100.0% (11 of 11 strings)

Translation: server-auth-16.0/server-auth-16.0-auth_admin_passkey
Translate-URL: https://translation.odoo-community.org/projects/server-auth-16-0/server-auth-16-0-auth_admin_passkey/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-auth-16.0/server-auth-16.0-auth_admin_passkey
Translate-URL: https://translation.odoo-community.org/projects/server-auth-16-0/server-auth-16-0-auth_admin_passkey/
Currently translated at 100.0% (3 of 3 strings)

Translation: server-auth-16.0/server-auth-16.0-auth_admin_passkey
Translate-URL: https://translation.odoo-community.org/projects/server-auth-16-0/server-auth-16-0-auth_admin_passkey/es/
Currently translated at 100.0% (3 of 3 strings)

Translation: server-auth-17.0/server-auth-17.0-auth_admin_passkey
Translate-URL: https://translation.odoo-community.org/projects/server-auth-17-0/server-auth-17-0-auth_admin_passkey/it/
In the res_users.py file, the `ignore_totp` session variable is now set based on the value of the `auth_admin_passkey_ignore_totp` configuration option. If the option is enabled, the `ignore_totp` session variable is set to True. This ensures that the `_mfa_url` method returns None when `ignore_totp` is True, effectively bypassing the 2FA check.
@BT-dlagin BT-dlagin force-pushed the 18.0-mig-auth_admin_passkey branch 3 times, most recently from 34b5faf to 37f98fc Compare December 20, 2024 13:50
@BT-dlagin BT-dlagin force-pushed the 18.0-mig-auth_admin_passkey branch from 37f98fc to fda78d0 Compare December 20, 2024 13:52
@BT-dlagin BT-dlagin merged commit ec97f9d into brain-tec:18.0.t154502 Dec 20, 2024
4 checks passed
@BT-dlagin BT-dlagin changed the title [18.0][MIG] auth_admin_passkey: Migration to 18.0 [t154502][18.0][MIG] auth_admin_passkey: Migration to 18.0 Dec 20, 2024
Copy link

codecov bot commented Dec 20, 2024

Codecov Report

Attention: Patch coverage is 91.07143% with 15 lines in your changes missing coverage. Please review.

Project coverage is 92.11%. Comparing base (8cba86b) to head (fda78d0).
Report is 94 commits behind head on 18.0.t154502.

Files with missing lines Patch % Lines
auth_admin_passkey/models/res_users.py 71.15% 10 Missing and 5 partials ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           18.0.t154502     #523      +/-   ##
================================================
- Coverage         92.26%   92.11%   -0.15%     
================================================
  Files                34       40       +6     
  Lines               517      685     +168     
  Branches             35       43       +8     
================================================
+ Hits                477      631     +154     
- Misses               27       37      +10     
- Partials             13       17       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.