Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[KeywordAlerts] Breaks custom emojis with keyword in the name #39

Open
ShortTimeNoSee opened this issue Jul 20, 2023 · 0 comments
Open

Comments

@ShortTimeNoSee
Copy link

Describe the bug

If an emoji's name (custom emojis confirmed, haven't tested with built-in emojis) contains a keyword (no regex set, just plain keyword) and the bold-first-instance setting is enabled, the custom emoji will be broken.

To Reproduce

  • Enable "Bold the first instance of a keyword in a message"
    Screenshot_20230719-204703.png
  • Have someone send a custom emoji that's name has the plain, non-regex keyword.

Expected behavior

Keep the emoji intact (while keeping the alert and mention (if mention setting is enabled), or ignore the keyword mention entirely (no alert or mention).

Priority

P3 or P4

Screenshots

Screenshot_20230719-204039~2.png

Mobile

  • Device: Moto E 5G (2022)
  • OS: Android 13
  • Screen size: 165.4 x 75.8 x 9.4 mm
  • Aliucord: 5CB4135
  • Discord: 126.21

Additional context

I am aware that the setting does state that "issues may arise" (and that you "do not care" 🥱), but I thought I'd submit this issue anyway.

There are workarounds using regex.

Example:

(?<!<:[\w_]*)(?:Nick)(?![\w_]*:[0-9]+>)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant