-
-
Notifications
You must be signed in to change notification settings - Fork 424
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] auth_oauht_multi_token: Migration to 17.0 #694
base: 17.0
Are you sure you want to change the base?
Conversation
Allow multiple oauth login at the same time.
* cleanup, improve, docstrings * add tests
Nothing special. Just making linters happy and splitting the readme. @Tecnativa TT25619
Otherwise lookup is slow when there are many users.
Otherwise you can't delete a user.
This is cosmetic only, because this field is not used when auth_oauth_multi_token is installed.
Currently translated at 100.0% (15 of 15 strings) Translation: server-auth-16.0/server-auth-16.0-auth_oauth_multi_token Translate-URL: https://translation.odoo-community.org/projects/server-auth-16-0/server-auth-16-0-auth_oauth_multi_token/it/
Can you please remove the pandoc-3.4-1-amd64.deb file. It looks like you missed the Edit: ah the step is no longer in the migration docs: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-17.0 |
@ap-wtioit now pandoc-3.4-1-amd64.deb removed |
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 is important to squash the *.deb removal commit with the commit, where the file was added. Otherwise this large file stays in the git repository.... in this case the commit needs to be squashed with "[IMP] auth_oauth_multi_token: pre-commit auto fixes"
There are also some more commits which should be squashed with the commit above:
Based on #678 and thanks to tb-flyt
Tried to follow instructions and hopefully I got it right 😃