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

[Streams] Significantly reduce number the quota usage for YouTube Data api #3237

Merged
merged 8 commits into from Feb 15, 2020
Merged

[Streams] Significantly reduce number the quota usage for YouTube Data api #3237

merged 8 commits into from Feb 15, 2020

Conversation

Drapersniper
Copy link
Contributor

@Drapersniper Drapersniper commented Dec 30, 2019

Type

  • Bugfix
  • Enhancement
  • New feature

Description of the changes

  1. Stop using the search endpoint...
  2. User the channels RSS feed to check the latest videos for less quota usage ( This can be changed to use the channel endpoing)
  3. Check for live streams every 5 minutes instead of 1 minute ([p]streamset timer was added so this can be customized down to 1 minute)
  4. reduce the number of repeated API calls by caching non-livestreams and skipping them in future calls.

@Flame442 Flame442 added the Type: Enhancement label Dec 30, 2019
Signed-off-by: Drapersniper <27962761+drapersniper@users.noreply.github.com>
Signed-off-by: Drapersniper <27962761+drapersniper@users.noreply.github.com>
Signed-off-by: Drapersniper <27962761+drapersniper@users.noreply.github.com>
Signed-off-by: Drapersniper <27962761+drapersniper@users.noreply.github.com>
Signed-off-by: Drapersniper <27962761+drapersniper@users.noreply.github.com>
Signed-off-by: Drapersniper <27962761+drapersniper@users.noreply.github.com>
Signed-off-by: Drapersniper <27962761+drapersniper@users.noreply.github.com>
* [Streams] Fix Twitch token for streamalert

* [Streams] Fix Twitch token for streamalert
palmtree5
palmtree5 previously requested changes Jan 17, 2020
redbot/cogs/streams/streamtypes.py Show resolved Hide resolved
@palmtree5 palmtree5 dismissed their stale review Feb 15, 2020

irrelevant, see #3487

Copy link
Member

@TrustyJAID TrustyJAID left a comment

Looks good to me.

@mikeshardmind mikeshardmind merged commit 04b5a5f into Cog-Creators:V3/develop Feb 15, 2020
2 checks passed
@mikeshardmind mikeshardmind added this to the 3.3.2 milestone Feb 15, 2020
@Drapersniper Drapersniper deleted the streams-api-fix branch Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Streams Type: Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants