fix: support emoji aliases like :smile: in PartialEmoji.from_str

[Lumabots]

Summary

Information

• This PR fixes an issue.
  Fixes discord.Emoji and discord.PartialEmoji didn't work in slash option. #1582
• This PR adds something new (e.g. new method or parameters).
• This PR is a breaking change (e.g. methods or parameters removed/renamed).
• This PR is not a code change (e.g. documentation, README, typehinting,
  examples, ...).

Checklist

• I have searched the open pull requests for duplicates.
• If code changes were made then they have been tested.
  • I have updated the documentation to reflect the changes.
• If type: ignore comments were used, a comment is also left explaining why.
• I have updated the changelog to include these changes.

[Lumabots]

fix of #1582

[Lumabots]

so my current fix is using the emoji lib : https://pypi.org/project/emoji/
how should i do, just add this to the requirement txt ?

[Paillat-dev]

@Lumabots What concerns me is that the emoji lib does not necessarily always include all of discord's nomenclature for markdown emojis. Notably, see this and this reddit threads.

Also, if this fixes #1582, please edit your pr description to add Fixes #1582 so that the pr is linked to the issue.

[Lumabots]

@Lumabots What concerns me is that the emoji lib does not necessarily always include all of discord's nomenclature for markdown emojis. Notably, see this and this reddit threads.

Also, if this fixes #1582, please edit your pr description to add Fixes #1582 so that the pr is linked to the issue.

I have been using this for years and I never really had any issue of missing emoji, but I'll try to do a test with every emoji of discord to see if there is lacking one.
Also in case there is lacking one do you have an alternative ?

[Lumabots]

Im thinking else I can do a json file with every discord emoji so I can just replace it

[Paillat-dev]

@Lumabots dmed you on discord a huge json from discord with all the emojis. Maybe it can be helpful to you in some way ? Specifically the nameToEmoji and emoji keys (I think you can ignore the other two). However i am not sure storing a json like that in the lib is the best strategy.

[NeloBlivion]

I'm not sure this is really ideal; users are expected to pass actual unicode emojis, not :name:

[Lumabots]

Lala added a bug tag in the related issue so I assumed it was

[Lumabots]

should be ready for review using Paillait Lib dismoji

[Dorukyum]

I'm strongly against adding a new dependency for this.

[Paillat-dev]

To be honest I agree that adding a dependency is not a good idea. To be clear, while this issue made me create that lib, I did not create it with the intent of it specifically be used in py-cord. Maybe a note should simply be added indicating to use a custom converter with a link to this pr &/ the issue.

An alternative could be to modify PartialEmoji to support storing an emoji exclusively based on a :some_emoji: string, however there wouldn't be anything to make sure that emoji is valid.

[Lumabots]

Why not just use ur system so we do a requests at first to get the emoji list from discord, and then just replace it directly ? So no need to install a new one

[Lumabots]

so i implemented a way to just download paillait file and then replace the emoji, i just need to find a way to load this file wihout blocking the bot. if you want to proceed differently tell me

[Paillat-dev]

@Lumabots cor security reasons it is not possible to have it download at startup. Instead, you would need to download the file and store it next to the python code and load from there, and the file should be updated periodically.

To open the file you can use pathlib like this:

import json
from pathlib import Path

EMOJIS_MAP_PATH = Path(__file__).parent / "emojis.json"

with EMOJIS_MAP_PATH.open("r", encoding="utf-8") as f:
    EMOJIS_MAP = json.loads(f.read())

[Lumabots]

should be ready for review

[Paillat-dev]

As long as including the json in the build is approved, this lgtm. However I have no idea wether there are some rules that would exclude it at build time so that should be verified.