Skip to content

[UHN-BS-36] Forgot password endpoint #81

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.

Sign up for GitHub

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

Jump to bottom
Open
mankitkong wants to merge 6 commits into
base: master
Choose a base branch
from
Open

[UHN-BS-36] Forgot password endpoint #81

mankitkong wants to merge 6 commits into from

Conversation

@mankitkong
Copy link
Contributor

@mankitkong mankitkong commented May 4, 2020

  • POST /reset-password
    • Requires username and phone field
    • Generates a random password of 5 length (cuz why not)
    • Texts using Twilio to phone field with temporary password
  • TextMessagingService -> for Twilio and other text messaging functionality
  • ResetPasswordService -> Forget password workflow related duties
  • validations.js -> for express-validator on request body/param requirements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@matthewleemle matthewleemle Awaiting requested review from matthewleemle

@andrchoi andrchoi Awaiting requested review from andrchoi

@mxnyee mxnyee Awaiting requested review from mxnyee

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mankitkong