• Type: Sub-task
• Resolution: Fixed
• Priority: L3 - Default
• Fix Version/s: 2.5.0-alpha1, 2.5.0
• Affects Version/s: None
• Component/s: frontend
• Labels:

  None

This is the controller panel for Smart Panels app

[OPT-2151] Spike: Request immutable data only once

Collapse comment: Sebastian Stamm added a comment - 12/Apr/19 3:26 PM

Sebastian Stamm added a comment - 12/Apr/19 3:26 PM

We only cache truly immutable data for now. Adding heuristics to cache mutable data would introduce too much complexity for too little benefit. omran.abazeed also had the idea to control caching via the cache-control response header, allowing the backend to control how the browser caches data and reducing frontend complexity. I like this idea and propose to discuss this in the next team meeting.

Expand comment: Sebastian Stamm added a comment - 12/Apr/19 3:26 PM

Sebastian Stamm added a comment - 12/Apr/19 3:26 PM We only cache truly immutable data for now. Adding heuristics to cache mutable data would introduce too much complexity for too little benefit. omran.abazeed also had the idea to control caching via the cache-control response header, allowing the backend to control how the browser caches data and reducing frontend complexity. I like this idea and propose to discuss this in the next team meeting.

Collapse comment: Sebastian Stamm added a comment - 17/Apr/19 1:22 PM

Sebastian Stamm added a comment - 17/Apr/19 1:22 PM

We agreed to do this caching via the browser mechanisms instead of using custom frontend code. I created OPT-2177 for this and reverted the changes for this ticket as they did not work in all cases.

Expand comment: Sebastian Stamm added a comment - 17/Apr/19 1:22 PM

Sebastian Stamm added a comment - 17/Apr/19 1:22 PM We agreed to do this caching via the browser mechanisms instead of using custom frontend code. I created OPT-2177 for this and reverted the changes for this ticket as they did not work in all cases.