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

dev/core#1919 - Missing resubscribe url in text/plain version of unsubscribe confirmation email #18015

Merged
merged 1 commit into from
Aug 2, 2020

Conversation

demeritcowboy
Copy link
Contributor

Overview

https://lab.civicrm.org/dev/core/-/issues/1919

  1. Go back in time about 10 years so that your email reader displays text-only versions of emails.
  2. Unsubscribe from a mailing.
  3. Read the email confirmation - it's missing the url to resubscribe at the end.

Another way to see this without needing time travel is to go to Administer - CiviMail - Headers,Footers.. and look at the one for unsubscribe and look in the text format section.

Before

You have been un-subscribed from the following groups: {unsubscribe.group}. You can re-subscribe by mailing {action.resubscribe} or clicking

After

You have been un-subscribed from the following groups: {unsubscribe.group}. You can re-subscribe by mailing {action.resubscribe} or clicking {action.resubscribeUrl}

Technical Details

ts in the xml was missing a parameter.

Comments

In the github UI it's hard to see but for civicrm_generated the change is at the end of the row for id 5.

@civibot
Copy link

civibot bot commented Jul 31, 2020

(Standard links)

@civibot civibot bot added the master label Jul 31, 2020
@seamuslee001
Copy link
Contributor

@demeritcowboy do we need to have an upgrade step associated with this?

@demeritcowboy
Copy link
Contributor Author

Oh, yes you're right, if they haven't modified already. Will do.

@eileenmcnaughton
Copy link
Contributor

test this please

@eileenmcnaughton
Copy link
Contributor

@seamuslee001 your comment has been addressed

@mattwire
Copy link
Contributor

mattwire commented Aug 2, 2020

This looks fine - @demeritcowboy It would really make sense to try and make these actual messagetemplates at some point - would make management of them much simpler.

@mattwire mattwire merged commit a12be26 into civicrm:master Aug 2, 2020
@eileenmcnaughton
Copy link
Contributor

I think @michaelmcandrew tried to do that once.

@demeritcowboy
Copy link
Contributor Author

Thanks!

@demeritcowboy demeritcowboy deleted the resubscribe-missing branch August 2, 2020 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants