Skip to content

[stable31] Fix: writing during short connection failures - #7096

Merged
juliusknorr merged 2 commits into
stable31from
backport/7093/stable31
Apr 11, 2025
Merged

[stable31] Fix: writing during short connection failures#7096
juliusknorr merged 2 commits into
stable31from
backport/7093/stable31

Conversation

@backportbot

@backportbotbackportbotBot commented Apr 11, 2025

Copy link
Copy Markdown

Backport of #7093

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Allow changing a flag with different delays:
* Switch it on slow.
* Switch it off fast.
Useful for errors - so they only show after a while but are resolved quickly.
Example:
const input = ref(false)
const { delayed } = useDelayedFlag(input)
...
input.value = true
// 5 seconds later delayed.value will be true
...
input.value = false
// 200 ms later delayed.value will be true
Signed-off-by: Max <max@nextcloud.com>
@backportbotbackportbotBot added bug Something isn't working 3. to review labels Apr 11, 2025
@backportbotbackportbotBot added this to the Nextcloud 31.0.3 milestone Apr 11, 2025
@max-nextcloud
max-nextcloudforce-pushed the backport/7093/stable31 branch from dc7029b to 94c5ff4CompareApril 11, 2025 08:53
@max-nextcloud
max-nextcloud marked this pull request as ready for review April 11, 2025 08:53
Delay the error message and allow editing for 5 seconds.
Also speed up the recovery by triggering push after a successful sync.
Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud
max-nextcloudforce-pushed the backport/7093/stable31 branch from 94c5ff4 to a133584CompareApril 11, 2025 08:55
@codecov

codecovBot commented Apr 11, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 52.04%. Comparing base (3070e24) to head (a133584).
Report is 3 commits behind head on stable31.

Additional details and impacted files
@@ Coverage Diff @@## stable31 #7096 +/- ##
============================================
+ Coverage 52.01% 52.04% +0.02% 
============================================
Files 286 287 +1 Lines 40675 40698 +23 Branches 789 794 +5 ============================================
+ Hits 21159 21182 +23 
Misses 19395 19395 Partials 121 121 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@juliusknorr
juliusknorr merged commit 16060b1 into stable31Apr 11, 2025
@juliusknorr
juliusknorr deleted the backport/7093/stable31 branch April 11, 2025 10:34
This was referenced Apr 15, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewbugSomething isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@max-nextcloud@juliusknorr