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.

Sign up for GitHub

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

Jump to bottom

[17.0][MIG] auth_admin_passkey #578

Merged
merged 79 commits into from
Nov 26, 2023
Merged

[17.0][MIG] auth_admin_passkey #578

merged 79 commits into from
Nov 26, 2023

Conversation

amkarthik
Copy link
Member

legalsylvain and others added 30 commits November 21, 2023 11:32
@legalsylvain @amkarthik
[ADD] new module 'auth_admin_passkey' that allow admin user (uid=1) t…
c293890 
…o login with any login and his password.
@legalsylvain @amkarthik
[IMP] move 'if' test to avoid useless call to check_credentials funct…
1e316af 
…ion if user_id == SUPERUSER_ID. (Courtesy of Stefan Rijnhart).
@legalsylvain @amkarthik
[IMP] 'auth_admin_passkey' : Manage the special case where an user ha…
eb8a8c5 
…s the same password as the admin user, sending a mail to admin user.
@legalsylvain @amkarthik
[ADD] 'auth_admin_passkey' : add '.pot' file.
a6229d2
@legalsylvain @amkarthik
[IMP] 'auth_admin_passkey' works now indepently from the order of the…
5d3dcdd 
… installation of modules 'auth_admin_passkey' and 'auth_crypt'. (Courtesy of Stefan Rijnhart).
@legalsylvain @amkarthik
[IMP] Emails are now translated.
f35408c
@legalsylvain @amkarthik
[REF] Pep8 and import conventions.
cecafd7
@legalsylvain @amkarthik
[FIX] res_config.py : incorrect call. self.pool('xxx') -> self.pool['…
1385314 
…xxx']

[REF] res_users.py : same call to self.pool in other res_users class.
@legalsylvain @amkarthik
[ADD] copyright information in all files.
b28ae43
@legalsylvain @amkarthik
[REF] Pep8.
b351612
@legalsylvain @amkarthik
[REF] change order in an algorithm;
3029cf7
@legalsylvain @amkarthik
[FIX] Incorrect import.
e55b445
@legalsylvain @amkarthik
[REF] typo.
277ca08
@legalsylvain @amkarthik
[FIX] translation.
17493a8
@amkarthik
Launchpad automatic translations update.
da34d65
@amkarthik
Launchpad automatic translations update.
c84b08c
@amkarthik
Launchpad automatic translations update.
ee38546
@legalsylvain @amkarthik
[ADD] Test OCA#6 to test the bug #1319391;
a158d88 
[ADD] Test OCA#1 to OCA#5 to avoid regression;
@legalsylvain @amkarthik
[FIX] bug #1319391;
cb4f3fe
@legalsylvain @amkarthik
[REF] fix comments;
50b4cfa
@amkarthik
Launchpad automatic translations update.
119111a
@legalsylvain @amkarthik
[REF] 'auth_admin_passkey': flake8.
3e01946
@sbidoul @amkarthik
move all modules to __unported__ on master branch and set installable…
ab7a597 
…=False
@anthony-muschang @amkarthik
[CHG] auth_admin_passkey: migration to 8.0
4630cdf
@anthony-muschang @amkarthik
[CHG] auth_admin_passkey: move description to README.md
a96ebac
@anthony-muschang @amkarthik
[CHG] auth_admin_passkey: rename README.md to README.rst
0954cef
@pedrobaeza @amkarthik
[IMP] auth_admin_passkey: Moved icon
322faff
@gurneyalex @amkarthik
Add OCA as author of OCA addons
21a3820 
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.
@oca-transbot @amkarthik
OCA Transbot updated translations from Transifex
facf721
@sbidoul @amkarthik
[UPD] prefix versions with 8.0
d0ae329
OCA-git-bot and others added 10 commits November 21, 2023 11:32
@OCA-git-bot @amkarthik
auth_admin_passkey 14.0.1.0.1
195c2f5
@AnizR @amkarthik
[IMP] auth_admin_passkey: pre-commit stuff
ff9e74b
@AnizR @amkarthik
[MIG] auth_admin_passkey: migration to 16.0
2b8c42a
@AnizR @amkarthik
[IMP] possibility to store password sha512 encrypted in config
6947f46
@amkarthik
[UPD] Update auth_admin_passkey.pot
6d19b14
@OCA-git-bot @amkarthik
[UPD] README.rst
240d682
@Ivorra78 @amkarthik
Translated using Weblate (Spanish)
cc9853c 
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/
@OCA-git-bot @amkarthik
[UPD] README.rst
306c22e
@weblate @amkarthik
Update translation files
7a3d785 
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/
@Ivorra78 @amkarthik
Translated using Weblate (Spanish)
9524c25 
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/
@pedrobaeza
Copy link
Member

[IMP] auth_admin_passkey: black, isort, prettier commit contains a .deb file that shouldn't be there, and the version should be changed in the other commit. Another extra thing is that the commit message is not correct. See https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-17.0

@amkarthik amkarthik force-pushed the 17.0-mig-auth_admin_passkey branch from 6c371aa to 7d9ca96 Compare November 21, 2023 07:29
@amkarthik
Copy link
Member Author

@pedrobaeza
Done with the changes.
Thanks

@pedrobaeza
Copy link
Member

/ocabot migration auth_admin_passkey

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Nov 21, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 21, 2023
Open
19 tasks
legalsylvain
legalsylvain approved these changes Nov 21, 2023
View reviewed changes
Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Diff review. LGTM. thanks !

atchuthan
atchuthan approved these changes Nov 21, 2023
View reviewed changes
Copy link
Member

@atchuthan atchuthan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review 👍

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@legalsylvain
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-578-by-legalsylvain-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c551151 into OCA:17.0 Nov 26, 2023
6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 32b0350. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@atchuthan atchuthan atchuthan approved these changes

@legalsylvain legalsylvain legalsylvain approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
17.0
Development

Successfully merging this pull request may close these issues.