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

emote_set_id in Get Channel Emotes is not properly set #413

Closed
iProdigy opened this issue Jun 9, 2021 · 2 comments
Closed

emote_set_id in Get Channel Emotes is not properly set #413

iProdigy opened this issue Jun 9, 2021 · 2 comments
Labels
product: api API endpoints in the "helix" namespace ticketed Has been given an internal tracking ticket

Comments

@iProdigy
Copy link

iProdigy commented Jun 9, 2021

Brief description
The emote_set_id for emotes returned in Get Channel Emotes is not accurate; currently it is simply being set to the id itself.

How to reproduce

  1. Query "Get Channel Emotes" with a broadcaster id (e.g. 207813352)

For example, here is an abridged emote example from the data response:

{id=300709075, name=hasKomrade, images={ ... }, tier=1000, emote_type=subscriptions, emote_set_id=300709075}

Notice that id == emote_set_id. This is repeated across the emotes in the response.

 

  1. To solidify the point, we can pass this emote_set_id to "Get Emote Sets"
{data=[{id=301266433, name=martioMaestria1, images={url_1x=https://static-cdn.jtvnw.net/emoticons/v1/301266433/1.0, url_2x=https://static-cdn.jtvnw.net/emoticons/v1/301266433/2.0, url_4x=https://static-cdn.jtvnw.net/emoticons/v1/301266433/3.0}, emote_type=subscriptions, emote_set_id=300709075, owner_id=411550143}]}

Notice that this response deals with a completely different broadcaster.

Expected behavior
emote_set_id should actually be set to the emote set id, not emote id.

@iProdigy iProdigy added the product: api API endpoints in the "helix" namespace label Jun 9, 2021
@iProdigy
Copy link
Author

iProdigy commented Jun 9, 2021

Looks like this issue was reproduced before me: Chatterino/chatterino2#2875 (comment)

@jbulava jbulava added the ticketed Has been given an internal tracking ticket label Jun 9, 2021
@jbulava
Copy link
Member

jbulava commented Jun 9, 2021

This fix is now in production.

@jbulava jbulava closed this as completed Jun 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product: api API endpoints in the "helix" namespace ticketed Has been given an internal tracking ticket
Projects
None yet
Development

No branches or pull requests

2 participants