Skip to content

Revert "LG-10886 remove ssn from flow session"#9209

Merged
aduth merged 1 commit intomainfrom
revert-9182-sonia-lg-10886-remove-ssn-from-flow-session
Sep 14, 2023
Merged

Revert "LG-10886 remove ssn from flow session"#9209
aduth merged 1 commit intomainfrom
revert-9182-sonia-lg-10886-remove-ssn-from-flow-session

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Sep 14, 2023

Reverts #9182

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.

@aduth aduth merged commit 4f0ad04 into main Sep 14, 2023
@aduth aduth deleted the revert-9182-sonia-lg-10886-remove-ssn-from-flow-session branch September 14, 2023 17:30
soniaconnolly added a commit that referenced this pull request Sep 18, 2023
Fixed 50/50 state issue by adding :ssn to pii going to proof resolution job in deploy 314.2.

This reverts commit 4f0ad04.
soniaconnolly added a commit that referenced this pull request Sep 18, 2023
* Revert "Revert "LG-10886 remove ssn from flow session (#9182)" (#9209)"

Fixed 50/50 state issue by adding :ssn to pii going to proof resolution job in deploy 314.2.

This reverts commit 4f0ad04.

* Don't use ||= when adding ssn to pii for proof resolution job

If someone updates their SSN, we want it to be updated for the proofing job, so use =
mitchellhenke pushed a commit that referenced this pull request Sep 18, 2023
* Revert "Revert "LG-10886 remove ssn from flow session (#9182)" (#9209)"

Fixed 50/50 state issue by adding :ssn to pii going to proof resolution job in deploy 314.2.

This reverts commit 4f0ad04.

* Don't use ||= when adding ssn to pii for proof resolution job

If someone updates their SSN, we want it to be updated for the proofing job, so use =
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