Skip to content

LG-15000: Clean up in_person_please_call#11663

Closed
matthinz wants to merge 8 commits intomainfrom
matthinz/15000-idv-please-call-part-3
Closed

LG-15000: Clean up in_person_please_call#11663
matthinz wants to merge 8 commits intomainfrom
matthinz/15000-idv-please-call-part-3

Conversation

@matthinz
Copy link
Contributor

(This PR is 3 of 3. It can't be merged until #11604 and #11662 are in production)

🎫 Ticket

Link to the relevant ticket:
LG-15000

🛠 Summary of changes

#11604 added a new idv_please_call email, which is identical to in_person_please_call.

#11662 started sending that email from other parts of IDV and updated the in-person code to send idv_please_call instead of in_person_please_call

This PR removes references to in_person_please_call.

- Change to idv_please_call to indicate it is used by idv generally
- Keep in_person_please_call as an alias on the theory that we need it for 1 deploy

[skip changelog]
I was having some trouble working with this, so I reworked it a little and added param docs
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.
This has been renamed to idv_please_call.

[skip changelog]
@matthinz
Copy link
Contributor Author

whoops, wrong base. closing

@matthinz matthinz closed this Dec 18, 2024
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