Skip to content

Lg 10361 retry verification after cancelling a barcode#9041

Merged
kbighorse merged 6 commits intomainfrom
LG-10361-Retry-Verification-After-Cancelling-a-Barcode
Aug 21, 2023
Merged

Lg 10361 retry verification after cancelling a barcode#9041
kbighorse merged 6 commits intomainfrom
LG-10361-Retry-Verification-After-Cancelling-a-Barcode

Conversation

@kbighorse
Copy link
Contributor

@kbighorse kbighorse commented Aug 18, 2023

🎫 Ticket

LG-10361 retry verification after cancelling a barcode

🛠 Summary of changes

This PR addresses an issue when the user creates a profile and receives a barcode and cancels the barcode. At this point the user is unable to retry verification.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • spec/features/idv/in_person_spec.rb

  • Create account, start IdV

  • Upload error yaml to get access to in person proofing

  • Go all the way through the process, including personal key, to receive a barcode

  • Cancel barcode

  • Expect to see Welcome screen and be able to complete remote proofing successfully

  • Expect step indicators to no longer be in person step indicator.

@kbighorse kbighorse requested a review from gina-yamada August 18, 2023 21:41
@kbighorse
Copy link
Contributor Author

tagging @18F/identity-joy-engineers @gina-yamada

@gina-yamada
Copy link
Contributor

I observed the redirect occurring after canceling a barcode and starting over. I observed the correct step indicator in the remote flow after visiting the IPP flow. (previously, the IPP step indicator was displaying in the remote flow after canceling a barcode) LGMT

@kbighorse kbighorse merged commit b08357b into main Aug 21, 2023
@kbighorse kbighorse deleted the LG-10361-Retry-Verification-After-Cancelling-a-Barcode branch August 21, 2023 18:35
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