Page MenuHomePhabricator

Cache clientside json for graph
Open, Stalled, Needs TriagePublic

Description

The API call that gets it doesn't have any caching parameter. It should be trivial to add those with a maxage parameter. Maybe don't add caching if the user is logged-in so that editors who modify the graph data see their updates immediately, but for unlogged visitors, the graph data should be cached for at the very least a few minutes.

Event Timeline

The api call is currently not in use, at the moment it is using the jsconfigvars to fetch the json for the graph. This might change again in the future.

@Jseddon is this ticket still relevant, post rewrite?

Jdlrobson changed the task status from Open to Stalled.Apr 25 2023, 12:02 AM

@Seddon: Could you please answer the last comment? Thanks in advance!