Skip to content

LG-11464: Redirect /verify/activated to personal key page if unackowledged#9586

Merged
matthinz merged 1 commit intomainfrom
matthinz/11464-personal-key-redirect
Nov 14, 2023
Merged

LG-11464: Redirect /verify/activated to personal key page if unackowledged#9586
matthinz merged 1 commit intomainfrom
matthinz/11464-personal-key-redirect

Conversation

@matthinz
Copy link
Contributor

🎫 Ticket

LG-11464

🛠 Summary of changes

If the user navigates to the /verify/activated URL but still have a personal_key in their idv_session (meaning they have not yet acknowledged it), redirect them back to the personal key page so they can acknowledge it.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Complete identity verification, but stop on the Personal key screen
  • Navigate to /verify/activated
  • Expect to be redirected back to the personal key screen

If the user has not acknowledged their personal key, don't show the "You already verified your identity" screen, but redirect them to ensure they get their personal key.

changelog: Bug Fixes, Identity verification, Redirect to personal key page rather than telling the user they already verified their identity.
@matthinz matthinz requested a review from a team November 13, 2023 23:42
Copy link
Contributor

@theabrad theabrad left a comment

Choose a reason for hiding this comment

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

LGTM. Tried it locally and it worked as intended.

@matthinz matthinz merged commit 40923fb into main Nov 14, 2023
@matthinz matthinz deleted the matthinz/11464-personal-key-redirect branch November 14, 2023 21:17
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