Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: /roadmap caching improvements #385

Open
SgtPooki opened this issue Oct 3, 2023 · 0 comments
Open

feat: /roadmap caching improvements #385

SgtPooki opened this issue Oct 3, 2023 · 0 comments

Comments

@SgtPooki
Copy link
Contributor

SgtPooki commented Oct 3, 2023

We have previously resolved some caching issues (i.e. #380), but one remains for /roadmap paths.

The current functionality works as follows:

  1. User requests roadmap view
  2. /roadmap request is made
  3. ServiceWorker has cache and returns it, but request waits until backend responds ("slow loading")

We should speed up responses for this path and also update the fix implemented by #383 is modified so that updated content can be retrieved.

Currently, the fix in #383 does not update the cache for /pendingChild paths unless the root roadmap request is not cached. Some additional optimization and tweaks need to be made here.

Some ideas:

  1. SW cache should only ever return immediately (do not keep request open while contacting backend).
    • Updating the cache should happen outside of main req-response workflow. We could use a queue of some sort to track urls returned from SW cache that we need to poll to check for updates, that are sent separately from user request.
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

No branches or pull requests

1 participant