Skip to content

Allow all unicode emojis#61

Merged
BenHall-1 merged 1 commit intoTicketsBot-cloud:masterfrom
biast12:allow-all-unicode-emojis
Nov 29, 2025
Merged

Allow all unicode emojis#61
BenHall-1 merged 1 commit intoTicketsBot-cloud:masterfrom
biast12:allow-all-unicode-emojis

Conversation

@biast12
Copy link
Copy Markdown
Contributor

@biast12 biast12 commented Nov 15, 2025

Description

Replaces custom emoji name-to-unicode mapping with direct Unicode emoji validation using UTF-8 checks in the backend and emoji-regex in the frontend. Removes unused emoji utility files and related initialization code, simplifying emoji handling and reducing maintenance overhead.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Improvement

Checklist

  • My code follows the style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Replaces custom emoji name-to-unicode mapping with direct Unicode emoji validation using UTF-8 checks in the backend and emoji-regex in the frontend. Removes unused emoji utility files and related initialization code, simplifying emoji handling and reducing maintenance overhead.
@biast12 biast12 requested a review from a team as a code owner November 15, 2025 00:48
Copy link
Copy Markdown
Member

@BenHall-1 BenHall-1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

@BenHall-1 BenHall-1 merged commit 2bdbb4f into TicketsBot-cloud:master Nov 29, 2025
@biast12 biast12 deleted the allow-all-unicode-emojis branch November 29, 2025 16:43
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

Successfully merging this pull request may close these issues.

3 participants