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

Go back to default YouTube cache #9372

Merged
merged 1 commit into from
Jul 14, 2023
Merged

Go back to default YouTube cache #9372

merged 1 commit into from
Jul 14, 2023

Conversation

PyvesB
Copy link
Member

@PyvesB PyvesB commented Jul 14, 2023

YouTube has finally granted the API quota increase.

I'm still trying to figure out what the new quota is, as the YouTube API team unhelpfully stated:

We have increased your Quota by 200,000. The total quota for xxxx is now 200,000 units per day

The request form also insisted on how much additional quota we wanted, our previous quota being 50,000, the new quota may actually be 250,000. Either way, we should have more than enough for many months to come. Let's set the cache back to its default value.

@github-actions
Copy link
Contributor

Messages
📖 ✨ Thanks for your contribution to Shields, @PyvesB!

Generated by 🚫 dangerJS against 01b0fc3

@chris48s
Copy link
Member

In https://console.cloud.google.com/apis/api/youtube.googleapis.com/quotas it says our limit is 200,000 queries per day.

We might actually want to settle on something a bit longer than the default but shorter than this. Lets see what happens to our usage when we deploy this anyway.

@chris48s chris48s added the service-badge New or updated service badge label Jul 14, 2023
@chris48s chris48s merged commit e7fc6d5 into master Jul 14, 2023
@chris48s chris48s deleted the youtube-cache branch July 14, 2023 14:42
@chris48s chris48s mentioned this pull request Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service-badge New or updated service badge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants