Custom emoji are not converted to text in short email notifications

email

(Michael Downey) #1

Continuing the discussion from Suppress images for short emails:

Reproduce:

  1. Configure Discourse so you get notifications for everything to ensure you’ll see the notification in question.
  2. Post a very short topic or reply with both a “standard” emoji e.g. :smile: and a custom emoji.

Expected behavior:

  • Both emoji are converted to text as described in the above-referenced change.

Actual behavior:

  • Only the “standard” emoji is converted to text; the custom emoji still appears as an image.

(Jeff Atwood) #2

Minor bug for this code @techapj, we need to check the Spam Assassin replacement regexes…


(Régis Hanol) #3

Thanks for reporting that @downey. I just pushed a fix :bee:


(Arpit Jalan) #4