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

Use django cache with large-image #32

Merged
merged 8 commits into from
Jun 17, 2022
Merged

Use django cache with large-image #32

merged 8 commits into from
Jun 17, 2022

Conversation

banesullivan
Copy link
Contributor

@banesullivan banesullivan commented Jun 16, 2022

requires girder/large_image#876

resolves #2

To do:

  • add some sort of test that will validate that large-image is really using the Django cache

@codecov
Copy link

codecov bot commented Jun 16, 2022

Codecov Report

Merging #32 (e404c8f) into main (e39d46c) will increase coverage by 0.05%.
The diff coverage is 83.72%.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   94.69%   94.75%   +0.05%     
==========================================
  Files          15       16       +1     
  Lines         622      648      +26     
==========================================
+ Hits          589      614      +25     
- Misses         33       34       +1     

@banesullivan banesullivan marked this pull request as ready for review June 17, 2022 15:06
@banesullivan banesullivan merged commit 5b51444 into main Jun 17, 2022
@banesullivan banesullivan deleted the feat/cache branch June 17, 2022 15:07
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.

Tie large-image into Django's cache
1 participant