-
-
Notifications
You must be signed in to change notification settings - Fork 36k
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
fix(user): Update email pre-confirm config #15985
fix(user): Update email pre-confirm config #15985
Conversation
@raisedadead updated the pull request. |
1 similar comment
@raisedadead updated the pull request. |
Earlier we did not need emails for everyone, this is not the case anymore. We now require emails, and all other auth mechanisms are now deprecated for new users.
@raisedadead updated the pull request. |
@raisedadead updated the pull request. |
@BerkeleyTrue I just talked with @raisedadead and he said this is ready to QA - could you take a quick look and comment or merge if it's ready? |
// work around should let us sneak by | ||
// see: | ||
// https://github.com/strongloop/loopback/issues/1137#issuecomment-109200135 | ||
delete User.validations.email; |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This request aims to make the following changes to the user model and email auth:
Refactor common code for creating verification tokens for update and login paths.This can be taken up in a separate pull request.Closes #15822