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(SyncService): handle failing connection gracefully #4990

Merged
merged 2 commits into from
Nov 14, 2023

Conversation

juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Nov 11, 2023

Backporting this specific commit to avoid random failures of cypress in https://github.com/nextcloud/text/pull/4990/files#diff-103df33bb085aaec0d6134982c312e96ecf58cf09335b9cf15964e17a45b4071R114

Signed-off-by: Max [email protected]

📝 Summary

  • Resolves: #

🖼️ Screenshots

🏚️ Before 🏡 After
B A

🚧 TODO

  • ...

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

Copy link

cypress bot commented Nov 11, 2023

1 flaky test on run #12359 ↗︎

0 150 2 0 Flakiness 1

Details:

[stable26] fix(SyncService): handle failing connection gracefully
Project: Text Commit: 62b6712395
Status: Passed Duration: 04:25 💡
Started: Nov 14, 2023 8:32 AM Ended: Nov 14, 2023 8:36 AM
Flakiness  cypress/e2e/sync.spec.js • 1 flaky test

View
Output

Test Artifacts
Sync > saves on close Screenshots

Review all test suite changes for PR #4990 ↗︎

@juliusknorr juliusknorr marked this pull request as ready for review November 13, 2023 17:58
@juliusknorr juliusknorr added bug Something isn't working 3. to review labels Nov 13, 2023
@juliusknorr juliusknorr changed the title fix(SyncService): handle failing connection gracefully [stable26] fix(SyncService): handle failing connection gracefully Nov 13, 2023
Also fixes the share cypress test.

Signed-off-by: Max <[email protected]>
@mejo- mejo- force-pushed the fix/stable26-docStateVersion branch from a73ef12 to 6164fb6 Compare November 14, 2023 08:06
Signed-off-by: nextcloud-command <[email protected]>
@mejo-
Copy link
Member

mejo- commented Nov 14, 2023

Waiting with merge due to rc phase of stable26

Hu, are we in RC phase for stable26 already?

@mejo- mejo- merged commit 83c4bb6 into stable26 Nov 14, 2023
32 checks passed
@mejo- mejo- deleted the fix/stable26-docStateVersion branch November 14, 2023 08:47
@juliusknorr
Copy link
Member Author

Yep, somehow thought I'd have seen the rc pr already merged, nevermind

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants