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.
mlaitinen
force-pushed
the
16.0-mig-auth_oauth_multi_token
branch
2 times, most recently
from
b099455 to
35637c6
Compare
|
I didn't touch |
|
/ocabot migration auth_oauth_multi_token |
pedrobaeza
left a comment
pedrobaeza
left a comment
There was a problem hiding this comment.
The README is regenerated on merge, yes.
|
|
||
| _name = "auth.oauth.multi.token" | ||
| _description = "OAuth2 token" | ||
| _description = "OAuth2 Token" |
There was a problem hiding this comment.
Don't change existing capitalization. It's not incorrect as it was. Same comment for the rest of this kind of changes.
There was a problem hiding this comment.
Is this an OCA policy? If yes, could you please send me a link? Only remark about capitalization I could find is here: CONTRIBUTING.rst.
This is the way how titles in English are capitalized, and also how the names of all models & fields in Odoo are formatted.
mlaitinen
force-pushed
the
16.0-mig-auth_oauth_multi_token
branch
from
35637c6 to
2c90556
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
Syncing from upstream OCA/server-auth (18.0)
7 participants
res.usersform by horizontally "unsqueezing" the list of auth tokens