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

CLDR-7405 Mail link to reset password #3764

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

srl295
Copy link
Member

@srl295 srl295 commented May 30, 2024

CLDR-7405

  • This PR completes the ticket.

ALLOW_MANY_COMMITS=true

srl295 added 3 commits May 28, 2024 19:44
- remove some dead code from LoginButton
- add a new ResetPassword and CldrLogin panels
- add a new option to the KeepLoggedInManager to generate a 1-hour token for use in email

At present the flow is that if you forgot your password, you will be given an option
to send a link in the mail. For smoketest, the link shows up on the console for testing.

Currently, the link takes you to a panel where you can remain logged into the Survey Tool.
In the future we can add logic to allow changing the password from there.
@srl295 srl295 self-assigned this May 30, 2024
@srl295
Copy link
Member Author

srl295 commented May 30, 2024

forgot_password_7405.mp4

demo of reset flow

srl295 added a commit to srl295/cldr that referenced this pull request Jun 4, 2024
 - seems to be creating excessive DB connections
srl295 added a commit that referenced this pull request Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant