Skip to content

LG-11230 Redirect browser back button to “your letter is on the way”#9633

Merged
theabrad merged 7 commits intomainfrom
abrad-lg-11230-deactivate-back-button-letter
Nov 22, 2023
Merged

LG-11230 Redirect browser back button to “your letter is on the way”#9633
theabrad merged 7 commits intomainfrom
abrad-lg-11230-deactivate-back-button-letter

Conversation

@theabrad
Copy link
Contributor

🎫 Ticket

LG-11230

🛠 Summary of changes

If a user recently requested a letter and reached the letter enqueued page if they pressed the back button they would be taken to the the Enter Code screen. We want them to only reach the enter code screen once they get a letter or start a new session so this redirects them back to the letter enqueued screen.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Go through the IdV verify by mail flow.
  • Once you reach the "letter enqueued" page press the back button.
  • Confirm that you stay on the letter enqueued page
  • Confirm that if you start a new session you reach the "Enter Code" page.

changelog: User-Facing Improvements, IdV Verify By Mail, redirect user
to letter_enqueued on back button press
@theabrad theabrad requested a review from a team November 21, 2023 17:29
theabrad and others added 3 commits November 21, 2023 15:09
request letter spec will redirect a user to the letter_enqueued page if
they re-request a letter. the spec now has a user sign out before they
can complete the enter code step.

Co-authored-by: Sonia Connolly <sonia.connolly@gsa.gov>
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, discussed the addition of a new before action in IdvStepConcern.

@theabrad theabrad merged commit 1e9102b into main Nov 22, 2023
@theabrad theabrad deleted the abrad-lg-11230-deactivate-back-button-letter branch November 22, 2023 15:40
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