Skip to content

LG-8392: Email user when OK'ing their ThreatMetrix status#7555

Merged
matthinz merged 3 commits intomainfrom
matthinz/8392-email-youre-ok
Jan 3, 2023
Merged

LG-8392: Email user when OK'ing their ThreatMetrix status#7555
matthinz merged 3 commits intomainfrom
matthinz/8392-email-youre-ok

Conversation

@matthinz
Copy link
Contributor

🎫 Ticket

LG-8392

🛠 Summary of changes

When operators manually OK a user's ThreatMetrix status, send the "You verified your identity" email.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Attempt to verify an account, but get blocked by ThreatMetrix
  • Run rake users:review:pass and provide your UUID when prompted
  • Verify you receive a "You verified your identity with Login.gov" email

@matthinz matthinz requested a review from a team December 29, 2022 21:13
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

When operators manually OK a user's ThreatMetrix status, send the "You verified your identity" email.

changelog: Internal, ThreatMetrix, Email when OK'ing user's ThreatMetrix status

For LG-8392
@matthinz matthinz force-pushed the matthinz/8392-email-youre-ok branch from 17fbbc1 to 43f8bfc Compare December 30, 2022 22:14
Copy link
Contributor

@theabrad theabrad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@matthinz matthinz merged commit 9330ed9 into main Jan 3, 2023
@matthinz matthinz deleted the matthinz/8392-email-youre-ok branch January 3, 2023 17:47
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