Skip to content

Deactivate frontend caching via the backend#1083

Merged
SteveMicroNova merged 1 commit intomainfrom
NoCaching
Feb 25, 2026
Merged

Deactivate frontend caching via the backend#1083
SteveMicroNova merged 1 commit intomainfrom
NoCaching

Conversation

@SteveMicroNova
Copy link
Contributor

What does this change intend to accomplish?

Deactivates caching of the frontend css and javascript files to avoid users being stuck with stale code, particularly fetch requests that no longer match backend endpoints

Will require users to clear cache one time post release but never again afterwards

Checklist

  • Have you tested your changes and ensured they work?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • If applicable, have you updated the CHANGELOG?
  • Does your submission pass linting & tests? You can test on localhost using ./scripts/test

@SteveMicroNova SteveMicroNova mentioned this pull request Feb 24, 2026
7 tasks
@SteveMicroNova SteveMicroNova linked an issue Feb 24, 2026 that may be closed by this pull request
@linknum23
Copy link
Contributor

How was this tested?

@SteveMicroNova
Copy link
Contributor Author

SteveMicroNova commented Feb 24, 2026

How was this tested?

image

I can see the hashes for the javascript and css bundles are when refreshing the frontend (see image) so I could see when they did or didn't change as I made changes to javascript before redeploying.

Before this change, the hash was the same every time unless deactivating my cache, after deploying this and deleting my cache it changed with every new deploy as it's meant to.

@linknum23
Copy link
Contributor

Can you validate this works on android/ios as well.

@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 90.90909% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 50.27%. Comparing base (7499989) to head (1d07124).
⚠️ Report is 89 commits behind head on main.

Files with missing lines Patch % Lines
amplipi/app.py 90.90% 1 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1083      +/-   ##
==========================================
- Coverage   50.67%   50.27%   -0.40%     
==========================================
  Files          40       41       +1     
  Lines        7154     7372     +218     
==========================================
+ Hits         3625     3706      +81     
- Misses       3529     3666     +137     
Flag Coverage Δ
unittests 50.27% <90.90%> (-0.40%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@linknum23 linknum23 left a comment

Choose a reason for hiding this comment

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

Lets see if this fixes our volume problems!

@SteveMicroNova SteveMicroNova merged commit 2a94606 into main Feb 25, 2026
3 checks passed
@SteveMicroNova SteveMicroNova deleted the NoCaching branch February 25, 2026 21:24
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.

Add "update browser cache" flag to frontend

3 participants