Skip to content

Comments

LG-9939 Addendum: Make /verify/doc_auth/:step redirect to /verify/welcome#8729

Merged
soniaconnolly merged 1 commit intomainfrom
sonia-lg-9939-fix-doc-auth-redirect
Jul 6, 2023
Merged

LG-9939 Addendum: Make /verify/doc_auth/:step redirect to /verify/welcome#8729
soniaconnolly merged 1 commit intomainfrom
sonia-lg-9939-fix-doc-auth-redirect

Conversation

@soniaconnolly
Copy link
Contributor

🎫 Ticket

LG-9939

🛠 Summary of changes

The previously added legacy route left the user on /verify/doc_auth/welcome which would give them a 404 when they click Continue.

Also moved redirect spec to spec/requests/redirects_spec.rb

📜 Testing Plan

  • Create account
  • Navigate to /verify/doc_auth/welcome
  • Expect to be redirected to/verify/welcome
  • Navigate to /verify/doc_auth/document_capture
  • Expect to be redirected to/verify/welcome
  • Click Continue
  • Expect to be on /verify/agreement

The previously added legacy route left the user on /verify/doc_auth/welcome
which would give them a 404 when they click Continue.

[skip changelog]

Co-authored-by: Matt Hinz <matt.hinz@gsa.gov>
@soniaconnolly soniaconnolly requested review from a team, jmhooper and matthinz July 6, 2023 22:37
@soniaconnolly soniaconnolly merged commit bcec644 into main Jul 6, 2023
@soniaconnolly soniaconnolly deleted the sonia-lg-9939-fix-doc-auth-redirect branch July 6, 2023 22:49
@aduth aduth mentioned this pull request Aug 11, 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