Skip to content

LG-11699/LG-11700: Allow verified user to repeat idv if they need biometric#10010

Merged
matthinz merged 7 commits intomainfrom
matthinz/11699-step-up-2-the-selfie
Feb 5, 2024
Merged

LG-11699/LG-11700: Allow verified user to repeat idv if they need biometric#10010
matthinz merged 7 commits intomainfrom
matthinz/11699-step-up-2-the-selfie

Conversation

@matthinz
Copy link
Copy Markdown
Contributor

@matthinz matthinz commented Jan 31, 2024

🎫 Ticket

Link to the relevant tickets:

🛠 Summary of changes

Refines "does this user need idv?" checks to take into account when users need to step up from verified to verified with biometric.

📜 Testing Plan

  1. Make sure you have doc_auth_selfie_capture_enabled: true in your application.yml
  2. Go through IdV and verify without a biometric
  3. Make a request from an SP with biometric comparison enabled
  4. Go back through IdV, this time verifying with a selfie
  5. Ensure you get redirected back to the SP
  6. Log in again from the SP with biometric comparison enabled and ensure you don't have to re-repeat the idv flow.

changelog: User-Facing Improvements, Identity verification, Allow previously verified users to repeat idv to get selfie verification.
(Need to clean up the document capture step)
Allow specifying that you want it to complete doc auth with a selfie image
@matthinz matthinz marked this pull request as ready for review January 31, 2024 19:28
@matthinz matthinz requested a review from a team January 31, 2024 19:28
@matthinz matthinz changed the title LG-11699: Allow verified user to repeat idv if they need biometric LG-11699/LG-11700: Allow verified user to repeat idv if they need biometric Jan 31, 2024
Copy link
Copy Markdown
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

@matthinz matthinz merged commit d20b3f6 into main Feb 5, 2024
@matthinz matthinz deleted the matthinz/11699-step-up-2-the-selfie branch February 5, 2024 18:03
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