Skip to content

Cache web cover width and height - #475

Merged
mjmusante merged 1 commit into
iftechfoundation:mainfrom
dfabulich:cache-web-cover-geometry
Sep 3, 2026
Merged

Cache web cover width and height#475
mjmusante merged 1 commit into
iftechfoundation:mainfrom
dfabulich:cache-web-cover-geometry

Conversation

@dfabulich

Copy link
Copy Markdown
Contributor

Fixes #474

I manually tested this by:

  • loading test data and switching to comp phase 5
  • I added logging to _read_web_cover_geometry to track hits and misses
  • I navigated to http://localhost:8080/ballot
    • In my Docker logs, I saw that the cache was missed
    • In Docker shell, I saw that entries/100/web_cover/geometry.txt had just been created
  • I refreshed the ballot
    • In my Docker logs, I saw that the cache was hit
  • I logged in as nobody@example.com and navigated to http://localhost:8080/entry/100/update, and uploaded a replacement image.
    • In Docker shell, I saw that entries/100/web_cover/geometry.txt had just been updated
  • I refreshed the ballot
    • In my Docker logs, I saw that the cache was hit

@mjmusante
mjmusante merged commit 8b95ecc into iftechfoundation:main Sep 3, 2026
1 check passed
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.

Cache cover width/height

2 participants