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.

Sign up for GitHub

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

Jump to bottom

Increase YouTube cache time from 24h to 48h #409

Merged
merged 1 commit into from
Jan 11, 2023
Merged

Increase YouTube cache time from 24h to 48h #409

merged 1 commit into from
Jan 11, 2023

Conversation

pajlada
Copy link
Member

@pajlada pajlada commented Jan 11, 2023

Pull request checklist:

  • CHANGELOG.md was updated, if applicable

Description

@pajlada pajlada enabled auto-merge (squash) January 11, 2023 10:33
@pajlada pajlada merged commit 487fb81 into master Jan 11, 2023
@pajlada pajlada deleted the chore/increase-youtube-cache-length branch January 11, 2023 10:39
@leon-richardt
Copy link
Contributor

Out of interest: What was the reason for this change? Did we hit the request limit on the YouTube API?

@pajlada
Copy link
Member Author

pajlada commented Jan 11, 2023

Yes, we hit the request limit. This is a short term fix. Preferably this value would be configurable in the config file but no time to do that now

@leon-richardt leon-richardt mentioned this pull request Jan 11, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
no changelog entry needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pajlada @leon-richardt