Skip to content

Cleanup ThreatMetrix CSP before_action name now that in_person SSN step is out of the FSM#9176

Merged
soniaconnolly merged 4 commits intomainfrom
sonia-cleanup-threatmetrix-before-action-name
Sep 8, 2023
Merged

Cleanup ThreatMetrix CSP before_action name now that in_person SSN step is out of the FSM#9176
soniaconnolly merged 4 commits intomainfrom
sonia-cleanup-threatmetrix-before-action-name

Conversation

@soniaconnolly
Copy link
Contributor

🛠 Summary of changes

Remove the Flow State Machine version of the ThreatMetrix CSP before_action because both the remote and in_person SSN steps are now out of the FSM.

📜 Testing Plan

  • After merging, deploy to staging
  • Create account, start from Sinatra app, go through remote IdV
  • On SSN step, confirm there are no permissions errors in Javascript console
  • Cancel and restart IdV
  • In DocumentCapture, upload special license image to trigger in person proofing
  • On in person SSN step, confirm there are no permissions errors in Javascript console

soniaconnolly and others added 4 commits September 8, 2023 09:54
because the ssn step is outside the FSM in person flow now.

Co-authored-by: Gina Yamada <gina.yamada@gsa.gov>

[skip changelog]
Co-authored-by: Jessica Dembe <jessica.dembe@gsa.gov>
@soniaconnolly soniaconnolly requested review from a team and gina-yamada September 8, 2023 18:41
@soniaconnolly soniaconnolly merged commit 1eac334 into main Sep 8, 2023
@soniaconnolly soniaconnolly deleted the sonia-cleanup-threatmetrix-before-action-name branch September 8, 2023 18:43
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