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

Emoji identifiers in headline markdown are not evaluated #33067

Open
func0der opened this issue Aug 15, 2024 · 2 comments
Open

Emoji identifiers in headline markdown are not evaluated #33067

func0der opened this issue Aug 15, 2024 · 2 comments
Labels
Tasked Added to the internal issue tracking type: bug

Comments

@func0der
Copy link

Description:

Emoji identifiers in headline markdown are not interpreted

Steps to reproduce:

  1. Write a new message
# :newspaper: This is a headline
  1. Hit 'Send'

Expected behavior:

I expect :newspaper: to show up as 📰

Actual behavior:

It shows up as :newspaper: This is a headline

image

Server Setup Information:

  • Version of Rocket.Chat Server: 6.9.2
  • License Type: Community
  • Number of Users: Dont know
  • Operating System: Linux
  • Deployment Method: Docker
  • Number of Running Instances: 1
  • DB Replicaset Oplog: -
  • NodeJS Version: docker
  • MongoDB Version: 5.0

Client Setup Information

  • Desktop App or Browser Version: Desktop App, Flatpak, 4.0.1
  • Operating System: Fedora

Additional context

If works if I paste in the emoji directly and not with its identifier

Relevant logs:

@reetp
Copy link

reetp commented Aug 17, 2024

Thanks for reporting.

As per the docs please always test on the latest version before reporting as Rocket.Chat is fast moving and things are often fixed in newer versions.

I'd also update your own version to the latest - there is a security fix in there IIRC.

https://github.com/RocketChat/Rocket.Chat/releases/tag/6.9.5

I have tested this on open.rocket.chat which is always 'latest' and I can see it occurring.

I'll mention it to the team.

@reetp reetp added Tasked Added to the internal issue tracking type: bug labels Aug 17, 2024
@yaswanthkosuru
Copy link

The ui renders based on md field.So i need to change Rocket.Chat/packages/message-parser/src/grammar.pegjs
to generate correct md field .
correct me if i am wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tasked Added to the internal issue tracking type: bug
Projects
None yet
Development

No branches or pull requests

3 participants