[Engine] Tenant image cache #2665
Ticket(s) / Related Merge Requests
- Issue card: #2665 (closed)
- Front MR: front!2472 (merged)
Summary of Changes
This change refactors the cache-busting system on asset images on multi-tenant networks.
Testing Considerations
Notes:
- Use https://c81e728d9d4c2f636f067f89cc14862c.bens-networks-3.oke.minds.io.
- You will need to log in as my network tenant admin (
admin
) - this is the same password and username as previously but if you need it again, DM me. - Note: query string format:
lastCache=xyz
Checklist:
-
The topbar nav icon has a timestamp appended to the query string - make a note of this value. -
On updating the square logo, the timestamp on the topbar logo changes, and the image changes. (local ts). -
On refresh, the timestamp should change again (server ts from the db - WILL be different from the local TS, should be greater than the first timestamp the topbar nav icon had). -
The favicon in the document head should have a query string with a cache busting timestamp. -
Update the favicon - the favicon should update - its TS should also update.
Deployment Considerations
Update the DB:
ALTER TABLE `minds_tenant_configs`
ADD last_cache_timestamp timestamp DEFAULT NULL
AFTER updated_timestamp;
Regression Scope
Multi-tenant asset images (topbar logo and favicon).
Platform Affected (web, mobile, etc)
Front-end.
Developer Testing Completed
Manual, unit (phpspec, karma/jasmine).
Screenshots / Screen Recording
Square logo
simplescreenrecorder-2023-11-07_10.18.01
Favicon
simplescreenrecorder-2023-11-07_10.19.42
Does this impact
-
Localization -
Dark/light mode -
Guest mode
Definition of Done Checklist
-
The Acceptance Criteria has been met -
Code is tested: Testing includes unit/spec, E2E/automated and manual testing -
Merge requests description has been filled out
Edited by Ben