Skip to content

LG-9571: Warn on MVA error#8482

Merged
matthinz merged 10 commits intomainfrom
matthinz/9571-aamva-timeouts
May 31, 2023
Merged

LG-9571: Warn on MVA error#8482
matthinz merged 10 commits intomainfrom
matthinz/9571-aamva-timeouts

Conversation

@matthinz
Copy link
Contributor

🎫 Ticket

LG-9571

🛠 Summary of changes

Adds a new warning screen shown to the user for certain types of AAMVA errors:

  • When the MVA is unavailable ('ExceptionId: 0001')
  • When there is a system error ('ExceptionId: 0002')
  • When the request to the MVA has timed out ('ExceptionId: 0047')

(This covers the set of AAMVA errors seen in prod for the last ~4 months.)

📜 Testing Plan

  1. Sign up and start identity verification
  2. For your ID, upload a YAML document including the following:
state_id_number: "mvatimeout"
state_id_type: "drivers_license"
state_id_jurisdiction: "WA" # Needs to be an AAMVA state
  1. You should get the "We are working to resolve an error" screen (below)
  2. You should be able to retry over and over again without being throttled

👀 Screenshots

If relevant, include a screenshot or screen capture of the changes.

English:

verify-session-errors-state_id_warning-en

Spanish:

verify-session-errors-state_id_warning-es

French:

verify-session-errors-state_id_warning-fr

@matthinz matthinz requested a review from a team May 24, 2023 22:56
@matthinz matthinz force-pushed the matthinz/9571-aamva-timeouts branch 2 times, most recently from f875cc9 to 4999318 Compare May 26, 2023 20:56
Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

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

I tried this out and everything worked great, except Exit Login.gov sent me to Cancel, which I wasn't expecting. I suppose if they go to /account they'll have to start over when they come back, but maybe the button should be clearer that it's Cancel and Exit Login.gov.

matthinz added 9 commits May 30, 2023 11:17
Tell the user that we're having trouble with AAMVA

changelog: User-Facing Improvements, IdV, Warn user when unable to communicate with MVAs.
For timeouts, unavailability, etc.
Get the step indicator displayed on some of these errors.
Related to adding the step indicator back -- needed an additional stub
@matthinz matthinz force-pushed the matthinz/9571-aamva-timeouts branch from 4999318 to eb4381e Compare May 30, 2023 18:39
This isn't a failure to proof, it's the user choosing to stop.
@matthinz
Copy link
Contributor Author

Good catch, you're right! I checked with @artfulaction and it sounds like we want to use the return_to_sp url here. Updated in 43d0d32.

Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@soniaconnolly soniaconnolly 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 a92bc0b into main May 31, 2023
@matthinz matthinz deleted the matthinz/9571-aamva-timeouts branch May 31, 2023 18:52
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