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

Implement access tokens #15

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

TheCricket
Copy link

Currently you're limited to 30 tickets a minute. You can get 800 tickets a minute by simply adding the token header.

…cription as a stream offline notification is just an empty body and has no personal data. By creating a variable in the callback url you can easily figure out whose data is whose
@DaniilShev DaniilShev requested review from DaniilShev and nic11 May 18, 2019 18:57
@nic11
Copy link
Member

nic11 commented May 27, 2019

Hello!

Thank you very much for sending us a pull request. Sorry you had to wait so much time, but sadly we didn't have enough time to take a look at the changes :(

I have a few edits right now. During the day, I'll look at the rest and test it out. After that, we'll be ready to merge this.

P.S. BTW, I've noticed you've made another fork with few more changes. It would be great if you send a PR for them too

#1)

Implement per subscription callbacks. This is necessary for mass subs…
@TheCricket
Copy link
Author

I added it to this one :)

@JMTK
Copy link

JMTK commented Sep 18, 2020

This will now permanently error via Twitch if you don't specify your access token.
#18

If this github repo isn't updated I can make a new NPM package that reflects this change

@nic11 nic11 mentioned this pull request May 6, 2021
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