-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add password email notice #94
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
This looks spot on so far, except a couple things I pointed out.
Maybe sometime tomorrow we can take 30 minutes or an hour to pair on automated testing for this. TDD works quite well with MBT (model-based testing) for making changes to an existing system - but unfortunately this isn't one of the models I wrote a test suite for, so there's no existing test to change 😔
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.
@tyrw I made the updates and this appears to work now, according to my understanding. Could you try it out at the Vercel deployment and make sure it meets your expectation? If it does, this is good to go!
I added a user to the database with no password ( However, the toolkit is sending with the |
Normal
Closes DEV-785
When a user attempts to log in with a password but does not have a password yet, display a notice for them to check their email.