Skip to content

Conversation

@dnplkndll
Copy link
Contributor

No description provided.

@dnplkndll dnplkndll changed the title [18.0][MIG] auth_oauth_multitoken [18.0][MIG] auth_oauth_multi_token: Migration to 18.0 Jan 17, 2025
@dnplkndll dnplkndll force-pushed the 18.0-mig-auth_oauth_multi_token branch 2 times, most recently from 7282a94 to a8955ff Compare January 17, 2025 21:43
@kobros-tech kobros-tech force-pushed the 18.0-mig-auth_oauth_multi_token branch from a8955ff to ca2abb9 Compare February 4, 2025 18:08
Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

Can you squash the migration commits?

Otherwise, looking good

Florent de Labarre and others added 21 commits April 25, 2025 14:53
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.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-auth-17.0/server-auth-17.0-auth_oauth_multi_token
Translate-URL: https://translation.odoo-community.org/projects/server-auth-17-0/server-auth-17-0-auth_oauth_multi_token/
@kobros-tech kobros-tech force-pushed the 18.0-mig-auth_oauth_multi_token branch from ca2abb9 to 05076f0 Compare April 25, 2025 12:09
@kobros-tech
Copy link
Contributor

@chienandalu

I guess the commits are squashed now, and we can go ahead.

Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

Ok for me then 👍

@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). 🤖

@pedrobaeza
Copy link
Member

Please remove [18.0] from migration commit message.

imagen

@kobros-tech kobros-tech force-pushed the 18.0-mig-auth_oauth_multi_token branch from 05076f0 to 112e1d4 Compare April 28, 2025 11:21
@kobros-tech
Copy link
Contributor

@pedrobaeza
already done

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-754-by-pedrobaeza-bump-nobump, awaiting test results.

@pedrobaeza
Copy link
Member

/ocabot migration auth_oauth_multi_token

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Apr 28, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 28, 2025
22 tasks
@OCA-git-bot OCA-git-bot merged commit 04c2cfe into OCA:18.0 Apr 28, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at f8ec949. 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.