Skip to content

LG-11203 add/remove step indicators#9425

Merged
soniaconnolly merged 5 commits intomainfrom
sonia-lg-11203-add-remove-step-indicators
Oct 23, 2023
Merged

LG-11203 add/remove step indicators#9425
soniaconnolly merged 5 commits intomainfrom
sonia-lg-11203-add-remove-step-indicators

Conversation

@soniaconnolly
Copy link
Contributor

@soniaconnolly soniaconnolly commented Oct 20, 2023

🎫 Ticket

LG-11203

🛠 Summary of changes

Add step indicator to VerifyInfo rate limit warning page.
Remove step indicator from VerifyInfo rate limit failure page.

📜 Testing Plan

  • Create account, start IdV
  • Use SSN not starting with 666 or 900
  • Click Continue on VerifyInfo
  • Confirm that warning screen has a step indicator
  • Repeat Try Again and Continue until rate limited
  • Confirm that failure screen does not have a step indicator

👀 Screenshots

Warning:

Warning

Failure:

Failure

Co-authored-by: John Maxwell <john.maxwell@gsa.gov>
@soniaconnolly soniaconnolly force-pushed the sonia-lg-11203-add-remove-step-indicators branch from 31030ce to 57f0168 Compare October 20, 2023 20:57
@soniaconnolly soniaconnolly requested a review from a team October 20, 2023 20:59
changelog: User-facing Improvements, Verify Info step rate limit pages, Add step indicator to warning page and remove step indicator from rate limited page
@soniaconnolly soniaconnolly force-pushed the sonia-lg-11203-add-remove-step-indicators branch from 57f0168 to 0d048a5 Compare October 20, 2023 21:01
@soniaconnolly soniaconnolly changed the title LG-11203 add remove step indicators LG-11203 add/remove step indicators Oct 20, 2023
soniaconnolly and others added 3 commits October 20, 2023 14:37
This avoids a 500 error if error screens with step indicators are accessed from hybrid flow.

Co-authored-by: Matt Hinz <matt.hinz@gsa.gov>
Co-authored-by: Alex Bradley <alexander.bradley@gsa.gov>
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 6bd4c57 into main Oct 23, 2023
@soniaconnolly soniaconnolly deleted the sonia-lg-11203-add-remove-step-indicators branch October 23, 2023 18:25
@mdiarra3 mdiarra3 mentioned this pull request Oct 26, 2023
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