Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[feature] use simple tokens for newsletter un/subscription mails #7446

Closed
wants to merge 1 commit into from
Closed

[feature] use simple tokens for newsletter un/subscription mails #7446

wants to merge 1 commit into from

Conversation

Zeromax
Copy link

@Zeromax Zeromax commented Nov 17, 2014

Just a small improvement for Newsletter subscription and unsubscription mails.

use the parseSimpleToken for e-mail text creation in newsletter unsubscription and subscription mails

@leofeyer
Copy link
Member

Fixed in c414564.

@leofeyer leofeyer closed this Nov 20, 2014
@Zeromax Zeromax deleted the feature--newsletter-simple-token branch November 20, 2014 20:12
@Zeromax
Copy link
Author

Zeromax commented Nov 20, 2014

@leofeyer thank you for implementation

@leofeyer
Copy link
Member

Please run a few test if you have time :)

@Zeromax
Copy link
Author

Zeromax commented Nov 20, 2014

Sure, I have tested my proposal, but I can overcopy it with yours and test again ;)
Will do this tomorrow.

@leofeyer
Copy link
Member

My code is almost the same as yours, so I don't expect any problems.

@Zeromax
Copy link
Author

Zeromax commented Nov 21, 2014

Yep. it works. only difference is the channels stuff, but I think this was an old relict ;)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants