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

LG-9974: Remove verified info accordion from reactivation #8532

Merged
merged 3 commits into from
Jun 5, 2023

Conversation

aduth
Copy link
Member

@aduth aduth commented Jun 2, 2023

🎫 Ticket

LG-9974

🛠 Summary of changes

Removes the "Your verified information" accordion from the account reactivation flow, for consistency with the identity verification flow changes merged in #8508.

📜 Testing Plan

  1. Go to http://localhost:3000
  2. Create an account
  3. Go to http://localhost:3000/verify
  4. Complete identity verification
    1. Make sure to copy / record the personal key
  5. Go to http://localhost:3000
  6. Click "Sign out"
  7. Click "Forgot your password?"
  8. Submit the email address of the user you created
  9. Reset the password
  10. Sign in
  11. Click "Reactivate your profile now"
  12. Click "I have my key"
  13. Submit the key you copied from the identity verification
  14. Observe the "Re-enter your Login.gov password" page no longer displays a "Your verified information" accordion

👀 Screenshots

Before After
image image

changelog: User-Facing Improvements, Account Reactivation, Update content on password reentry step
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aduth aduth merged commit 92cde75 into main Jun 5, 2023
@aduth aduth deleted the aduth-lg-9974-rm-reactivate-pii branch June 5, 2023 14:01
@solipet solipet mentioned this pull request Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants