Skip to content

LG-11221 Personal key doesn't change on page refresh#9363

Merged
soniaconnolly merged 3 commits intomainfrom
sonia-lg-11221-consistent-personal-key
Oct 12, 2023
Merged

LG-11221 Personal key doesn't change on page refresh#9363
soniaconnolly merged 3 commits intomainfrom
sonia-lg-11221-consistent-personal-key

Conversation

@soniaconnolly
Copy link
Contributor

🎫 Ticket

LG-11221

🛠 Summary of changes

Previously, when the user refreshed the Personal Key page, a new key was generated each time. This could prevent the user from recovering their account if they saved the previous key and didn't notice it had changed. Now, the personal key remains the same when the page is refreshed.

📜 Testing Plan

  • Go through IdV up to personal key page
  • Refresh page
  • Expect key to be unchanged.

changelog: Bug Fixes, Identity Verification Personal Key, show same Personal Key when page is refreshed
@soniaconnolly soniaconnolly requested review from a team and jmhooper October 11, 2023 21:45
code = subject.idv_session.personal_key

get :show
get :show
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, I would have had to write an expensive feature test for this.

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

@soniaconnolly soniaconnolly merged commit 0f1a167 into main Oct 12, 2023
@soniaconnolly soniaconnolly deleted the sonia-lg-11221-consistent-personal-key branch October 12, 2023 20:37
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.

4 participants