sdlink icon indicating copy to clipboard operation
sdlink copied to clipboard

[Feature]: Support Emoji translation from MC chat -> Discord chat

Open CorneliusCornbread opened this issue 8 months ago • 9 comments

🚀 Description

Currently, typing out :emoji: in MC chat and having a emoji in Discord with the exact same name does not convert to that. This means mods like Emojiful work going from Discord -> MC but not vice versa.

Image

Image

💡 Use Case

Better integration with mods like Emojiful

CorneliusCornbread avatar Apr 30 '25 20:04 CorneliusCornbread

As far as I am aware, emojiful does NOT support custom emojis going from mc -> discord, only discord -> mc if you create a custom data pack for it.

We already fully support emojiful as far as possible

hypherionmc avatar May 01 '25 12:05 hypherionmc

Found this too: https://github.com/InnovativeOnlineIndustries/Emojiful/issues/132

hypherionmc avatar May 01 '25 12:05 hypherionmc

As far as I am aware, emojiful does NOT support custom emojis going from mc -> discord, only discord -> mc if you create a custom data pack for it.

We already fully support emojiful as far as possible

Right but if you could convert from the :emoji: format to discords emoji that'd be sick. DiscordSRV supports it.

CorneliusCornbread avatar May 01 '25 21:05 CorneliusCornbread

Right but if you could convert from the :emoji: format to discords emoji that'd be sick

We already do this, but can't do it for custom emojis, because Emojiful is missing the critical data needed for this to work. The dev of emojiful has confirmed as much. We are working with him to get it supported.

The issue is that discord does NOT use the same :emoji: format for custom emotes, but rather uses <a:emoji:9350350325703753> for them

hypherionmc avatar May 01 '25 21:05 hypherionmc

https://i.imgur.com/ZCpvSfJ.jpeg

hypherionmc avatar May 01 '25 21:05 hypherionmc

Right but if you could convert from the :emoji: format to discords emoji that'd be sick

We already do this, but can't do it for custom emojis, because Emojiful is missing the critical data needed for this to work. The dev of emojiful has confirmed as much. We are working with him to get it supported.

The issue is that discord does NOT use the same :emoji: format for custom emotes, but rather uses <a:emoji:9350350325703753> for them

Well the way DiscordSRV does it is it seems to look up the emoji name and convert it for you. Maybe they're querying the servers they're connected to do so?

CorneliusCornbread avatar May 02 '25 01:05 CorneliusCornbread

Just to add here as well, I mentioned this on discord:

Well the way DiscordSRV does it is it seems to look up the emoji name and convert it for you. Maybe they're querying the servers they're connected to do so?

Yes, this is possible, but, you would be limited to the emojis in your own discord server, since currently SDL can only function with a single discord server.

You will also run into the issue where a custom emote, and built-in emote can share the same name. How would cases like that be handled? Do we always favor the custom emote, or always favor the built-in one? Either one we pick will have someone complaining about it.

I marked this as on hold, because emojiful is considering adding the stuff we need to make this work properly, and I would like to have official, proper support as well

hypherionmc avatar May 13 '25 16:05 hypherionmc

You will also run into the issue where a custom emote, and built-in emote can share the same name. How would cases like that be handled? Do we always favor the custom emote, or always favor the built-in one? Either one we pick will have someone complaining about it.

If someone who owns a server duplicates the name of a custom emote with a standard emote they could literally just change the name of the emote, it sounds to me like a user error issue rather than anything else.

I marked this as on hold, because emojiful is considering adding the stuff we need to make this work properly, and I would like to have official, proper support as well

Fair enough, full integration would likely work best tbh.

CorneliusCornbread avatar May 13 '25 21:05 CorneliusCornbread

This is now partially supported in 3.3.1 (at least as best as possible, until emojiful adds the required data)

hypherionmc avatar May 16 '25 08:05 hypherionmc