Skip to content
This repository was archived by the owner on Aug 20, 2025. It is now read-only.

cache_implementation: test#64

Closed
DanielElisenberg wants to merge 2 commits intomainfrom
cache_implementation
Closed

cache_implementation: test#64
DanielElisenberg wants to merge 2 commits intomainfrom
cache_implementation

Conversation

@DanielElisenberg
Copy link
Contributor

Should write some unit tests etc.
Quick implementation to ask for everyones opinion on this as a quick fix

The latest version is always cached,
the last request is always cached,
draft is never cached.

@DanielElisenberg DanielElisenberg requested a review from a team as a code owner June 3, 2025 10:27
Co-authored-by: Linn Bjørnerud <125447659+linnbjornerud@users.noreply.github.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 3, 2025

Copy link
Contributor

@linnbjornerud linnbjornerud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this looks like a very simple and nice quick fix 💯

Copy link
Contributor

@haraldsvik haraldsvik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! should save a lot of reading and processing time!

@DanielElisenberg
Copy link
Contributor Author

#65 easier implementation here

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants