-
Notifications
You must be signed in to change notification settings - Fork 32
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
Clarify at login that a new account will be created #48
Comments
If a user logs into an account which is not yet connected to an existing user account (i.e. a new account would be created), we could add an additional page that does something like the following:
The second part might not be easy to implement because we will have to temporarily save the data from {Provider X} somewhere, but it should be possible with the session. |
Since this issue requires some changes in application flow (as mentioned in my previous comment), I would suggest moving it to 0.5.0 (or maybe to a 0.4.1). |
I wonder if this is really necessary. It already says on the login page:
The only use case I can think of is when a user logs in with a different provider than previously and now has two accounts. However, how often does that happen, and is it really such a big problem? I would vote for not changing this. Maybe we can simply add another sentence like "To connect an additional provider to your existing account, please log into that existing account first." |
Not required as accounts get merged if used with same identity. |
Specifically mention that the user first has to log into the existing account before connecting an additional identity. The purpose is to prevent users from accidentally creating two separate accounts instead of one account with two identities.
The text was updated successfully, but these errors were encountered: