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

Announcements: Fix being unable to react with the keycap number sign emoji #22231

Merged

Conversation

kescherCode
Copy link
Contributor

#⃣

This bug is caused by the emoji consisting of:
U+23 #
U+FE0F
U+20E3

Because it starts with a #, it's interpreted as an anchor link, which is not passed to the API. Therefore, the API sees no emoji to react with and answers correctly with a 404.

See also: glitch-soc#1980 (comment)

#⃣

This bug is caused by the emoji consisting of:
U+23 #
U+FE0F
U+20E3  ⃣

Because it starts with a #, it's interpreted as an anchor link, which is not passed to the API. Therefore, the API sees no emoji to react with and answers correctly with a 404.
@Gargron Gargron merged commit 04c611d into mastodon:main Dec 15, 2022
@kescherCode kescherCode deleted the announcement-reaction-keycap-number-sign branch December 15, 2022 17:00
neatchee pushed a commit to neatchee/mastodon that referenced this pull request Dec 16, 2022
…#22231)

#⃣

This bug is caused by the emoji consisting of:
U+23 #
U+FE0F
U+20E3  ⃣

Because it starts with a #, it's interpreted as an anchor link, which is not passed to the API. Therefore, the API sees no emoji to react with and answers correctly with a 404.
ClearlyClaire pushed a commit to glitch-soc/mastodon that referenced this pull request Dec 17, 2022
nametoolong pushed a commit to nametoolong/nuage that referenced this pull request Jan 12, 2023
…#22231)

#⃣

This bug is caused by the emoji consisting of:
U+23 #
U+FE0F
U+20E3  ⃣

Because it starts with a #, it's interpreted as an anchor link, which is not passed to the API. Therefore, the API sees no emoji to react with and answers correctly with a 404.
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