Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LG-15000: Send "Please call" email from other parts of IDV #11662

Draft
wants to merge 4 commits into
base: matthinz/15000-idv-please-call-part-1
Choose a base branch
from

Conversation

matthinz
Copy link
Member

@matthinz matthinz commented Dec 18, 2024

(This PR is 2 of 3. It can't be merged until #11604 has been deployed to production)

ALSO, turns out this one will depend on some changes over in #11665 landing first

🎫 Ticket

Link to the relevant ticket:
LG-15000

🛠 Summary of changes

Updates the non-IPP IDV flows to send the "Please Call" email

📜 Testing Plan

Phone verification

  1. Run through IDV, selecting phone verification. Ensure you are flagged by ThreatMetrix.
  2. Confirm you receive the "Please Call" email at the end of the flow
  3. Confirm you do not receive the "You've been verified" email at the end of the flow

GPO

  1. Run through IDV, selecting verify by mail. Ensure you are flagged by ThreatMetrix.
  2. Confirm you receive the "Please Call" email after you enter your code
  3. Confirm you do not receive the "You've been verified" email after you enter your code

@matthinz matthinz requested a review from a team December 18, 2024 00:13
@matthinz matthinz force-pushed the matthinz/15000-idv-please-call-part-2 branch 2 times, most recently from 6360574 to 10eecd3 Compare December 18, 2024 05:01
@matthinz matthinz marked this pull request as draft December 18, 2024 06:23
@matthinz matthinz removed the request for review from a team December 18, 2024 06:23
This is the exact same email, content-wise, just with a new name to reflect the fact that it is used throughout idv.

[skip changelog]
Many clients won't display inline images referenced by URL for privacy reasons.
@matthinz matthinz force-pushed the matthinz/15000-idv-please-call-part-2 branch from 10eecd3 to 51cc5a3 Compare December 18, 2024 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant