@electric-paper opened this Pull Request on July 20th 2018

fixes #12954

@fdellwing commented on July 20th 2018 Contributor

I'm not an english native, but shouldn't it be You are currently not opted out.?

@electric-paper commented on July 24th 2018

Of course, you're right. Changed it to You are currently not opted out.
My apologies.

@tsteur commented on July 24th 2018 Member

Looks good 👍 Any chance we can change the name of the translation key itself? Eg to "YouAreNotOptedOut" and remove "YouAreOptedIn"?

This will be needed so it gets translated in all other languages. Otherwise translators wouldn't notice that the English translation changed and they wouldn't be requested to translate it.

@electric-paper commented on July 31st 2018

Changed the key to YouAreNotOptedOut
I'm not quite sure about the failing checks though, seems to be a normal thing?

@diosmosis commented on August 8th 2018 Member

I think the text here could be shortened to just "You are not opted out." 'currently' seems unnecessary here. What does everyone think?

@mattab commented on August 8th 2018 Member

sounds good @diosmosis

Powered by GitHub Issue Mirror