Skip to content

LG-12377 Remove 404 action for State ID Controller/FSM removal#11243

Merged
jennyverdeyen merged 2 commits intomainfrom
jverdeyen/LG-12377-fsm-state-id-removal-404-action
Sep 18, 2024
Merged

LG-12377 Remove 404 action for State ID Controller/FSM removal#11243
jennyverdeyen merged 2 commits intomainfrom
jverdeyen/LG-12377-fsm-state-id-removal-404-action

Conversation

@jennyverdeyen
Copy link
Contributor

@jennyverdeyen jennyverdeyen commented Sep 16, 2024

🎫 Ticket

Link to the relevant ticket:
LG-12377

🛠 Summary of changes

This removes the before action that causes the State ID controller to show a 404 when the in_person_state_id_controller_enabled feature flag is turned off.

📜 Testing Plan

Both URLs work with the controller state enabled:

  • Ensure the environment variable in_person_state_id_controller_enabled is set to false in your application.yml and restart the application.
  • Go through the IPP flow
  • At the state ID page, change the url from /verify/in_person/state_id to /verify/in_person_proofing/state_id
  • Observe the state ID page loads and doesn't show a 404 error.

Disabling the controller variable to simulate 50/50 state:

  • Ensure the environment variable in_person_state_id_controller_enabled is set to true in your application.yml and restart the application.
  • Go through the IPP flow
  • At the state ID page go back to application.yml and change the in_person_state_id_controller_enabled to false, and restart the application.
  • Refresh the state ID page.
  • Observe the state ID page loads and doesn't show a 404 error.

Enabling the controller variable to simulate 50/50 state:

  • Ensure the environment variable in_person_state_id_controller_enabled is set to false in your application.yml and restart the application.
  • Go through the IPP flow
  • At the state ID page go back to application.yml and change the in_person_state_id_controller_enabled to true, and restart the application.
  • Refresh the state ID page.
  • Observe the state ID page loads and doesn't show a 404 error.

👀 Screenshots

Before: Screenshot 2024-09-16 at 3 26 20 PM
After: Screenshot 2024-09-16 at 3 27 15 PM

changelog: Internal, In-person proofing, Remove 404 action for State ID Controller/FSM removal
@jennyverdeyen jennyverdeyen requested review from a team and eileen-nava and removed request for eileen-nava September 16, 2024 20:10
@gina-yamada
Copy link
Contributor

gina-yamada commented Sep 17, 2024

Nice job Jenny! Good update on those 50/50 specs. We had not implemented those specs on previous steps. Thanks for the before/after screenshots. I included them too from my testing.

Screenshot 2024-09-17 at 2 26 15 PM Screenshot 2024-09-17 at 2 29 07 PM

@gina-yamada gina-yamada self-requested a review September 17, 2024 18:48
@gina-yamada
Copy link
Contributor

gina-yamada commented Sep 17, 2024

@jennyverdeyen Before smashing merge- might be good to confirm flag is disabled in production on AWS

@jennyverdeyen jennyverdeyen merged commit 3caa856 into main Sep 18, 2024
@jennyverdeyen jennyverdeyen deleted the jverdeyen/LG-12377-fsm-state-id-removal-404-action branch September 18, 2024 14:24
AShukla-GSA pushed a commit that referenced this pull request Sep 30, 2024
* LG-12377 Remove 404 action for State ID Controller/FSM removal

changelog: Internal, In-person proofing, Remove 404 action for State ID Controller/FSM removal
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.

3 participants