-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Registration with email verification process #49
Comments
Hey ! Can you assign me this issue ? |
We don't assign issues as we find it discourages others from fixing/implementing changes. |
Is the goal to replace username-based login with email authentication, so we can then implement the registration process with email verification ? |
It could be done without changing the username-based login. The goal here is to validate someone email. But I think we should use an email for authentification too, as most of users are used to login/register this way, plus it's already a unique identifier. They can update their username once registered on their profile if necessary. |
I've created an initial PR that adds an email field for users, which we can use as a foundation for further iterations. |
We should wait for #48 to be merged.
Currently we use a basic
username
for login, we should use anemail
.Migrations,
seed-database.ts
and tester helper must be updated before proceeding.The text was updated successfully, but these errors were encountered: