-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Create an administrator in new tenants #5048
Conversation
0f70971
to
7c1792a
Compare
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.
Interesting approach. I've left some comments!
7c1792a
to
4c0ba4c
Compare
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.
Much better! 🎉 I've left a couple of minor comments.
9fcd0c6
to
4e1bcf0
Compare
Looks like we've got a couple of flaky tests, including the one we've added in this pull request 😌. Just a theory: maybe the second one failed because the first one failed and some data leaked between tests 🤔. No idea why the first one (which is the one we've added) failed, though. The screenshot seems to show that clicking the "sign in" link resulted in a request that took too long, but I don't see a reason why that would happen. |
4e1bcf0
to
a689638
Compare
a689638
to
cba0f13
Compare
cba0f13
to
9dfae2d
Compare
Co-Authored-By: Senén Rodero <[email protected]>
9dfae2d
to
951eec7
Compare
References
Related PR: #4030
Objectives
When an admin creates a new tenant, we automatically add him as admin of the newly created tenant. The admin that is created is already automatically confirmed as it is a copy of an existing user with the same password.
Visual Changes
Notes
Add this behaviour to the multitenancy documentation.