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

[stable26] fix(conflict): resolve quickly with fast sync. #4127

Merged
merged 2 commits into from
May 5, 2023

Conversation

juliushaertl
Copy link
Member

Backport of #4076

Fixes #4075.

* Speed up the sync for one request AFTER the force save.
  Otherwise the `PollingBackend` will poll only once every 5 seconds
  when only one user is connected.
* The `$syncService.startSync()` had no effect
  as it tried to reconnect an already connected backend.

Signed-off-by: Max <max@nextcloud.com>
@juliushaertl juliushaertl added bug Something isn't working 3. to review labels May 5, 2023
@juliushaertl juliushaertl added this to the Nextcloud 26.0.2 milestone May 5, 2023
@cypress
Copy link

cypress bot commented May 5, 2023

1 flaky tests on run #9679 ↗︎

0 142 1 0 Flakiness 1

Details:

[stable26] fix(conflict): resolve quickly with fast sync.
Project: Text Commit: e2f0b35db5
Status: Passed Duration: 04:01 💡
Started: May 5, 2023 3:06 PM Ended: May 5, 2023 3:10 PM
Flakiness  cypress/e2e/sync.spec.js • 1 flaky test

View Output Video

Test Artifacts
Sync > recovers from a lost connection Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@max-nextcloud
Copy link
Collaborator

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@juliushaertl juliushaertl merged commit a7ed120 into stable26 May 5, 2023
@delete-merged-branch delete-merged-branch bot deleted the backport/4076/stable26 branch May 5, 2023 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants