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

Add Caching for faster response time. #2859

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Add Caching for faster response time. #2859

wants to merge 11 commits into from
+1,236 −19

Conversation

ildyria
Copy link
Member

@ildyria ildyria commented Dec 28, 2024 •
edited
Loading

Probably one of the biggest speed up available to anonymous users.
Will allow to use redis.

@ildyria ildyria changed the title ⚠️ Add Caching for faster response time. 🚧 Add Caching for faster response time. Dec 28, 2024
@ildyria ildyria changed the title 🚧 Add Caching for faster response time. 🚧 Add Caching for faster response time. Dec 28, 2024
@ildyria ildyria added the Review: medium Medium review expected: not many files, some attention to details required. label Dec 30, 2024
@ildyria ildyria requested a review from d7415 December 30, 2024 01:07
@ildyria ildyria changed the title 🚧 Add Caching for faster response time. Add Caching for faster response time. Dec 30, 2024
@ildyria ildyria marked this pull request as ready for review December 30, 2024 01:08
@codecov Codecov
Copy link

codecov bot commented Dec 30, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 59.52381% with 102 lines in your changes missing coverage. Please review.

Project coverage is 83.48%. Comparing base (47a8e6f) to head (dd6cf93).
Report is 1 commits behind head on master.

Additional details and impacted files

@ildyria ildyria added this to the 6.3 milestone Jan 3, 2025
@ildyria ildyria force-pushed the caching-responses-serverside branch from dc2beac to ab43e2e Compare January 10, 2025 01:00
@tinohager
Copy link
Contributor

@d7415 Hello Martin, I would be delighted if you could take a look here :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@d7415 d7415 Awaiting requested review from d7415

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Review: medium Medium review expected: not many files, some attention to details required.
Projects
None yet
Milestone
6.3
Development

Successfully merging this pull request may close these issues.
2 participants
@ildyria @tinohager