HAWK
(Hawk)
September 25, 2017, 1:19am
1
I have a report of someone that has been trying to update their email templates but the changes are not saving.
So far it has happened with both the Account Created and the Invite Mailer templates.
When they click “Save Changes” the “Saved!” message is returned but the changes are lost when the screen is refreshed.
It is happening in both Firefox and Safari.
I can’t repro it here or on try.discourse.org , but @erlend_sh didn’t you deal with the same thing last week?
3 Likes
Pad_Pors
(Pad Pors)
September 25, 2017, 5:40am
2
some similar issues here:
I changed the subject for the “User Posted PM” email template to [%{email_prefix}] [PM] %{username} sent you a private message but it is ignored and is still using [%{email_prefix}] [PM] %{topic_title}
Using 1.8.0.beta9. Editing other email template subject lines seems to work, this one is ignored.
Also, the line To unsubscribe from these emails, click here. is always added at the end of the message, regardless of what I put in the template body. Can we fix that too and allow us to change it (…
https://meta.discourse.org/t/translation-of-email-templates-does-not-get-saved-if/65204?u=pad_pors
erlend_sh
(Erlend Sogge Heggen)
September 25, 2017, 10:28am
3
Yeah, I’ve encountered this and received some reports about it. I think I was able to reproduce once and then it was seemingly working again.
1 Like
HAWK
(Hawk)
September 25, 2017, 4:22pm
4
@jomaxro Will you please see if you can repro it when you get a sec?
1 Like
jomaxro
(Joshua Rosenfeld)
September 25, 2017, 4:24pm
5
If I’m not mistaken @erlend_sh repro’d this the other day. Will add to my list.
3 Likes
We need this bug fixed, @gerhard can you have a look?
3 Likes
gerhard
(Gerhard Schlager)
September 27, 2017, 9:40pm
9
I fixed this today. You should get an error when you try to save an email template with missing interpolation keys.
https://github.com/discourse/discourse/commit/1a378126259b1461bb935d1385778842d33b088a
Bonus: Changing an email template will create an entry in the Staff Log. That didn’t happen before.
While testing this I also noticed that reverting changes to an email template or site text can result in a “translation missing” error. I think I found a solution for that as well, but I’m not comfortable merging it in until someone else from the team takes a look at it.
@Pad_Pors Unfortunately it’s currently not possible to omit interpolation keys.
7 Likes
gerhard
(Gerhard Schlager)
Closed
September 29, 2017, 9:40pm
10
This topic was automatically closed 2 days after the last reply. New replies are no longer allowed.