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

New emojis not supported in v5 #1020

Open
benwick opened this issue Feb 6, 2023 · 3 comments
Open

New emojis not supported in v5 #1020

benwick opened this issue Feb 6, 2023 · 3 comments
Labels
🐛 Bug Something isn't working 📦 message-parser

Comments

@benwick
Copy link

benwick commented Feb 6, 2023

Description:

After switching to Rocket.Chat v5, ZWJ emoji sequences are not supported anymore: https://emojipedia.org/emoji-zwj-sequence/

Steps to reproduce:

  1. Try to write a message and pick an ZWJ sequence emoji with the system emoji picker, not the Rocket.Chat emoji picker (on windows for example by pressing win+dot)
  2. send the message

Expected behavior:

The picked emoji is shown.

Actual behavior:

The sequence of emojis is shown.
image

Server Setup Information:

  • Version of Rocket.Chat Server: 5.4.2
  • Operating System: Debian 11
  • Deployment Method: Snap
  • Number of Running Instances: 1
  • NodeJS Version: 14.19.3
  • MongoDB Version: 4.4.15

Client Setup Information

  • Desktop App or Browser Version: Newest Electron and Android Apps
  • Operating System: Windows11 and Android12
@roshangeorge97
Copy link

please assign me this issue

@benwick benwick changed the title Emoji ZWJ Sequence not supported in v5 New emojis not supported in v5 Apr 11, 2023
@benwick
Copy link
Author

benwick commented Apr 11, 2023

@hugocostadev So this is not a problem at all?

@tassoevan tassoevan transferred this issue from RocketChat/Rocket.Chat Apr 11, 2023
@tassoevan tassoevan added 🐛 Bug Something isn't working 📦 message-parser labels Apr 11, 2023
@hugocostadev
Copy link
Contributor

@hugocostadev So this is not a problem at all?

It's a problem, this need be to implemented in our message-parser grammar at Fuselage repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working 📦 message-parser
Projects
None yet
Development

No branches or pull requests

4 participants