Notification error


(R Zarni) #1

at this time, I have launcher rebuild app

now,

Notification error.

image


#2

We are having the same problem with our language, English seems to work…


(Janno Liivak) #3

It seems that problem is with @eviltrout commit:


(Robin Ward) #4

Are there any errors in the logs? I did change the translations for notifications to remove hard coded HTML, but I’m not even seeing the text there.


#5

I don’t know if these problems are even related but they both appeared after updating discourse today.


(Janno Liivak) #6

I don’t understand … what text you don’t see? You don’t see the same problem when you change from Englist to something else … Spain for example?


(Janno Liivak) #7

If it’s any help here’s rendered HTML for non-English:

and here for English:


(Robin Ward) #8

I hadn’t tried it out yet – I was just looking for more data before checking out the branch again. I can reproduce it now and am looking for a fix that can last until things are translated properly.


(R Zarni) #9

I am now back off using the beta channel in containers/app.yml

:joy::joy:


#10

Is it safe to downgrade?


(R Zarni) #11

Looks like there is no problem.


(Robin Ward) #12

This should fix it up until the other languages are translated:


Notifications in user profile are broken
(Janno Liivak) #13

Could you point us to the right direction … which translations exacty are you talking about? I tried to search Transifex but can’t figure out what to search for :blush:


(Robin Ward) #14

I’m not sure how often Transifex updates from master, but the keys that need to have the HTML removed are in this commit:


(Gerhard Schlager) #15

I think about once or twice a day…


(Jeff Atwood) #16