Skip to content

Type, document, and refactor Sync, port tests to Mocha #1331

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

Open
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

raucao
Copy link
Member

@raucao raucao commented Jun 8, 2025

Includes some small other improvements. However, this PR is mostly about properly typing and documenting sync, as well as making the functions more readable by using async/await in many places.

Also adds a new event called "sync-started", when a periodic/background sync has started (but not when outgoing changes are synced immediately).

refs #1251 (where I added sub-issues that are all easier to tackle with these changes)
closes #1173
closes #839

P.S.: I rebased the branch as much as possible before opening the PR. But there are too many conflicts when trying to consolidate the rest even more.

raucao added 6 commits June 9, 2025 08:30
Use _rs_init instead of custom prototype properties
Only log the message as debug log, without the stack trace

refs #839
In order for devs not having to wait feature loading before managing
access and caching settings, we need those classes to be instantiated
onthe prototype already
@raucao
Copy link
Member Author

raucao commented Jun 9, 2025

FYI, I opened #1334 against this one, because it requires the new cachinglayer test suite. Can re-open against master later, if it doesn't get merged here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Add sync-started event Suppress SyncErrors when in offline mode
1 participant