Skip to content

[Engine] Tenant image cache #2665

Ben requested to merge feat/tenant-img-cache-e2665 into master

Ticket(s) / Related Merge Requests

Summary of Changes

This change refactors the cache-busting system on asset images on multi-tenant networks.

Testing Considerations

Notes:
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

Merge request reports