Custom emoji are not converted to text in short email notifications

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.
4 Likes

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

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

https://github.com/discourse/discourse/commit/13e5c026431c617e0803196931aaf23edb16c085

4 Likes