# Custom emoji are not converted to text in short email notifications

**URL:** https://meta.discourse.org/t/custom-emoji-are-not-converted-to-text-in-short-email-notifications/25685
**Category:** Bug
**Tags:** email
**Created:** [25. Februar 2015 um 16:56 UTC](https://meta.discourse.org/t/custom-emoji-are-not-converted-to-text-in-short-email-notifications/25685 "2015-02-25T16:56:03Z")
**Posts on this page:** 4
**Page:** 1

<div class="post-metadata">

### Author: ![downey](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/downey/32/166878_2.png) [@downey](https://meta.discourse.org/u/downey)
#### Post date: [25. Februar 2015 um 16:56 UTC](https://meta.discourse.org/t/custom-emoji-are-not-converted-to-text-in-short-email-notifications/25685/1 "2015-02-25T16:56:03Z")

</div>

Continuing the discussion from [Suppress images for short emails](https://meta.discourse.org/t/suppress-images-for-short-emails/20235/7):

> [@Suppress images for short emails](https://meta.discourse.org/t/suppress-images-for-short-emails/20235/7):
>
> Okay, as per the feedback, I have done following [updates](https://github.com/discourse/discourse/pull/2805/files):
> 
> - Only Avatar and Emoji images will be suppressed from email
> - Avatar image will be replaced with their relevant title text, e.g. ❤ will be replaced by `:heart:`
> - Short email length is configurable in site settings, default will be `2800`

**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. 😄 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.

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [26. Februar 2015 um 04:09 UTC](https://meta.discourse.org/t/custom-emoji-are-not-converted-to-text-in-short-email-notifications/25685/2 "2015-02-26T04:09:49Z")

</div>

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

---

<div class="post-metadata">

### Author: ![zogstrip](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/zogstrip/32/512781_2.png) [@zogstrip](https://meta.discourse.org/u/zogstrip)
#### Post date: [26. Februar 2015 um 11:51 UTC](https://meta.discourse.org/t/custom-emoji-are-not-converted-to-text-in-short-email-notifications/25685/3 "2015-02-26T11:51:34Z")

</div>

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

[https://github.com/discourse/discourse/commit/13e5c026431c617e0803196931aaf23edb16c085](https://github.com/discourse/discourse/commit/13e5c026431c617e0803196931aaf23edb16c085)

---

<div class="post-metadata">

### Author: ![techAPJ](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/techapj/32/342990_2.png) [@techAPJ](https://meta.discourse.org/u/techAPJ)
#### Post date: [26. Februar 2015 um 12:31 UTC](https://meta.discourse.org/t/custom-emoji-are-not-converted-to-text-in-short-email-notifications/25685/4 "2015-02-26T12:31:53Z")

</div>


