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

Mailing Unsubscribe Form: Show if email is already unsubscribed #21174

Merged
merged 1 commit into from
Sep 9, 2021

Conversation

mattwire
Copy link
Contributor

Overview

This cleans up messaging on the unsubscribe form so it is clearer (at least in english) what the current status is.

user@example.org is unsubscribed > user@example.org has been unsubscribed successfully
user@example.org has been unsubscribed > user@example.org has already been unsubscribed

Before

Click link in email:

Unsubscribe form shown.

Click unsubscribe:
Message is user@example.org is unsubscribed

Click link in email again:

Message is user@example.org has been unsubscribed.

After

Click link in email:
image

Click unsubscribe:
image
*Note: ignore the mailing error - that's related to #21173 and the confirmation message here is now clearer because the unsubscribe WAS successful but the email confirming that unsubscribe failed to send.

Message is user@example.org has been unsubscribed successfully.

Click link in email again:
image

Message is user@example.org has already been unsubscribed.

Technical Details

Comments

@civibot
Copy link

civibot bot commented Aug 18, 2021

(Standard links)

@civibot civibot bot added the master label Aug 18, 2021
@agileware-justin
Copy link
Contributor

Great PR @mattwire

Would really like to improve the user experience in this area, have raised a Gitlab for a discussion, https://lab.civicrm.org/dev/user-interface/-/issues/42

@wmortada
Copy link
Contributor

wmortada commented Sep 6, 2021

This makes sense to me. Thanks @mattwire

@mlutfy mlutfy changed the title Show if email is already unsubscribed on unsubscribe form Mailing Unsubscribe Form: Show if email is already unsubscribed Sep 7, 2021
@mlutfy
Copy link
Member

mlutfy commented Sep 7, 2021

Breaks translations, but worth it imho.

@mattwire Could you resolve the conflicts? (doing it via the Github IDE gives me rather odd results)

@mattwire mattwire force-pushed the unsubscribemessaging branch from 8d8c37d to 0bb34e3 Compare September 7, 2021 12:08
@mattwire
Copy link
Contributor Author

mattwire commented Sep 7, 2021

@mlutfy I've fixed the merge conflicts

@demeritcowboy demeritcowboy merged commit 8b7bcfc into civicrm:master Sep 9, 2021
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.

5 participants