HAWK
(Hawk)
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 לייקים
Pad_Pors
(Pad Pors)
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)
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
HAWK
(Hawk)
25 בספטמבר, 2017, 4:22pm
4
@jomaxro Will you please see if you can repro it when you get a sec?
לייק 1
jomaxro
(Joshua Rosenfeld)
25 בספטמבר, 2017, 4:24pm
5
If I’m not mistaken @erlend_sh repro’d this the other day. Will add to my list.
3 לייקים
We need this bug fixed, @gerhard can you have a look?
3 לייקים
gerhard
(Gerhard Schlager)
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 לייקים
gerhard
(Gerhard Schlager)
נסגר ב־
29 בספטמבר, 2017, 9:40pm
10
This topic was automatically closed 2 days after the last reply. New replies are no longer allowed.