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

Don't cache errors #328

Closed
wants to merge 68 commits into from
Closed

Conversation

Razzmatazzz
Copy link
Member

When the graphql response results in an unexpected error, don't cache that response.

thaddeus and others added 26 commits August 10, 2024 10:42
Don't cache zero-length responses
implement cache update cooldown
Pull updates from worker repo
@Razzmatazzz Razzmatazzz requested a review from a team as a code owner August 29, 2024 21:38
@Razzmatazzz Razzmatazzz deleted the no-cache-error branch August 29, 2024 21:41
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.

2 participants