Skip to content

Comments

LG-6401: Add warning alert to IdV verify step#6512

Merged
aduth merged 38 commits intomainfrom
LG-6401-warning-alert-to-verify-info
Jun 24, 2022
Merged

LG-6401: Add warning alert to IdV verify step#6512
aduth merged 38 commits intomainfrom
LG-6401-warning-alert-to-verify-info

Conversation

@peggles2
Copy link
Contributor

@peggles2 peggles2 commented Jun 21, 2022

Why: To allow the user the option to return to the document capture step to retake photos in case the uneditable information on the verify step is incorrect.

Screenshot:

localhost_3000_verify_doc_auth_verify

@aduth aduth force-pushed the LG-6401-warning-alert-to-verify-info branch from c518120 to 7704bec Compare June 22, 2022 14:04
@aduth aduth changed the title Lg 6401 warning alert to verify info LG-6401: Add warning alert to IdV verify step Jun 22, 2022
@aduth aduth requested a review from a team June 22, 2022 19:06
@aduth aduth marked this pull request as draft June 23, 2022 13:05
@aduth
Copy link
Contributor

aduth commented Jun 23, 2022

Putting this back into draft to iterate a bit more on the non-GET link behavior a bit more, since I discovered we'll need something very similar for LG-6402 with troubleshooting options that needs to be more detached from the button component.

Edit: Implemented across 6b0cc8e, fe18687, cfc9b6d.

@aduth aduth marked this pull request as ready for review June 23, 2022 13:46
Copy link
Contributor

@solipet solipet left a comment

Choose a reason for hiding this comment

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

LGTM! Tested with the following image yml file and got the alert:

document:
  type: license
  first_name: Susan
  last_name: Smithwrong
  middle_name: Q
  address1: 1 Microsoft Way
  address2: Apt 3
  city: Bayside
  state: NY
  zipcode: "11364"
  dob: 10/06/1938
  phone: +1 314-555-1212
failed_alerts:
  - name: 2D Barcode Read
    result: Attention

@aduth aduth merged commit 46182cf into main Jun 24, 2022
@aduth aduth deleted the LG-6401-warning-alert-to-verify-info branch June 24, 2022 13:02
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.

3 participants