Skip to content

LG-9066 Remove the FSM SSN step#7901

Merged
jmhooper merged 14 commits intomainfrom
jmhooper-remove-ssn-step
Mar 3, 2023
Merged

LG-9066 Remove the FSM SSN step#7901
jmhooper merged 14 commits intomainfrom
jmhooper-remove-ssn-step

Conversation

@jmhooper
Copy link
Contributor

We are working on retiring the FSM and have recently put work into moving users to a new controller for entering their SSN. This works follows up on that by removing the old SSN step and the code that directed users their.

We are working on retiring the FSM and have recently put work into moving users to a new controller for entering their SSN. This works follows up on that by removing the old SSN step and the code that directed users their.

changelog: Improvements, FSM Retirements, The old FSM step was removed and supporting code was also removed now that all users should be sent to the newly built SSN controller.
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.

Looks good so far!

Note: newly merged in_person VerifyInfoController also has a reference to this feature flag.

@jmhooper jmhooper marked this pull request as ready for review March 3, 2023 14:46
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.

Looks great! Thanks for grinding through all this.

@jmhooper jmhooper merged commit 42a4db7 into main Mar 3, 2023
@jmhooper jmhooper deleted the jmhooper-remove-ssn-step branch March 3, 2023 17:15
@jmdembe jmdembe mentioned this pull request Mar 9, 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