Skip to content
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

[backend] Don't let the user use an alternative login method based on the same email #151

Open
pfortuna opened this issue Apr 1, 2020 · 6 comments

Comments

@pfortuna
Copy link
Contributor

pfortuna commented Apr 1, 2020

When a user logs in with e.g. Google and is using a specific email e.g. [email protected], if we previously had a registration using another login method with the same email, we should reject that registration and an error should be returned that asks the user to use the same login method he used before.

@pfortuna pfortuna added UIUX This issue is waiting for UIUX work release issues that will be released next labels Apr 3, 2020
@pfortuna
Copy link
Contributor Author

pfortuna commented Apr 3, 2020

@geiras We need design of the error message /dialog.

@pfortuna pfortuna assigned pfortuna and geiras and unassigned pfortuna Apr 3, 2020
@geiras
Copy link

geiras commented Apr 5, 2020

Would be a possibility to remove the button of the wrong sign-in method after the warning message?

We already have design for error messages. Can we use the same, @pfortuna?

@pfortuna
Copy link
Contributor Author

pfortuna commented Apr 5, 2020

Would be a possibility to remove the button of the wrong sign-in method after the warning message?
Yes this is a possibility. And once we know, we also set a cookie to remember the correct login method. But sometimes we don't know that until we check the backend and return an error.

We already have design for error messages. Can we use the same, @pfortuna?
yes we can :-) can you post it here for facilitating?

@pfortuna
Copy link
Contributor Author

pfortuna commented Apr 5, 2020

Perfect ! moving this to TODO!

@geiras
Copy link

geiras commented Apr 5, 2020

This way the user can login with the correct method in the same popup where we show the error.

@pfortuna pfortuna removed the UIUX This issue is waiting for UIUX work label Apr 5, 2020
@pfortuna pfortuna removed the release issues that will be released next label Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants