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

Add new email content if user is authenticating to manage their subscriptions #1175

Merged
merged 1 commit into from Mar 20, 2020

Conversation

@huwd
Copy link
Contributor

huwd commented Mar 19, 2020

Trello: https://trello.com/c/QV5aaDq1/37-update-content-to-existing-manage-your-subscription-email

What

If the destination is /manage/authenticate, we can use a different subject line and body

Why

We're getting a lot of user confusion about the unsubscription flow. This is a first attempt at clarifying the content to see if it helps

@huwd huwd force-pushed the update-content-for-unsubscription-emails branch from aa25335 to 5659409 Mar 19, 2020
@huwd huwd requested a review from theseanything Mar 19, 2020
@huwd huwd force-pushed the update-content-for-unsubscription-emails branch from 5659409 to 568e4fd Mar 19, 2020
@huwd huwd force-pushed the update-content-for-unsubscription-emails branch 2 times, most recently from a60b1c3 to 2b4b857 Mar 19, 2020
…riptions

If the destination is /manage/authenticate, we can use a different subject line and body
@huwd huwd force-pushed the update-content-for-unsubscription-emails branch from 2b4b857 to 9a5a333 Mar 20, 2020
@huwd huwd requested a review from theseanything Mar 20, 2020
@huwd huwd merged commit ef027e3 into master Mar 20, 2020
2 checks passed
2 checks passed
continuous-integration/jenkins/branch This commit looks good
Details
continuous-integration/jenkins/security No security issues found
Details
@huwd huwd deleted the update-content-for-unsubscription-emails branch Mar 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.