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

feat: Caching improvements #194

Merged
merged 3 commits into from
Dec 2, 2024
Merged

feat: Caching improvements #194

merged 3 commits into from
Dec 2, 2024

Conversation

johnjcsmith
Copy link
Contributor

  • Auth context cache wasn't actually being set.
  • Update createCache to consult local node-cache and remote Redis

@johnjcsmith johnjcsmith requested a review from nadeesha December 2, 2024 10:24
@johnjcsmith johnjcsmith force-pushed the feat/caching-improvements branch from 2d7fedb to aa5aa3a Compare December 2, 2024 10:40
@johnjcsmith @nadeesha
Update cache.ts
30c2cf9 
Co-authored-by: Nadeesha Cabral <[email protected]>
@johnjcsmith johnjcsmith merged commit 76655de into main Dec 2, 2024
23 checks passed
@johnjcsmith johnjcsmith deleted the feat/caching-improvements branch December 2, 2024 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nadeesha nadeesha nadeesha left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@johnjcsmith @nadeesha