Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Sync v2 performance, initial sync limit=1 #9299

Closed
wants to merge 5 commits into from
Closed

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Sep 20, 2022

Known issues:

  1. Only ensures a minimum of a single previewable event has been fetched rather than a handful (e.g. 5-10)
  2. Doesn't apply any smart sorting as to what order to load them in (e.g. visible first)
  3. Doesn't persist work, as js-sdk only persists accumulated sync responses, meaning that on refresh all this data will need re-fetching.
  4. No unit tests

Here's what your changelog entry will look like:

✨ Features

  • Sync v2 performance, initial sync limit=1 (#9299).

@t3chguy t3chguy added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Sep 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants