Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Jul 27, 2025

Backport of #7501

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.

@backportbot backportbot bot added this to the Nextcloud 31.0.8 milestone Jul 27, 2025
@mejo- mejo- force-pushed the backport/7501/stable31 branch from 4fe5451 to 2f3e74c Compare July 30, 2025 09:15
@mejo- mejo- marked this pull request as ready for review July 30, 2025 09:16
@codecov
Copy link

codecov bot commented Jul 30, 2025

Codecov Report

❌ Patch coverage is 22.22222% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.55%. Comparing base (3b5907a) to head (8d03b62).
⚠️ Report is 9 commits behind head on stable31.

Files with missing lines Patch % Lines
src/views/RichWorkspace.vue 0.00% 14 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##           stable31    #7506   +/-   ##
=========================================
  Coverage     51.54%   51.55%           
=========================================
  Files           292      292           
  Lines         41472    41470    -2     
  Branches        875      875           
=========================================
+ Hits          21377    21378    +1     
+ Misses        19974    19971    -3     
  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.

Awaiting the request for the file lead to race conditions:
* Enter directory with rich workspace, loading is triggered.
* Navigate to directory without rich workspace.
* Loading finishes - rich workspace was shown.

Also clean up some minor things:
* `this.creating` was not used at all.
* The Editor does not emit `error`.

Signed-off-by: Max <max@nextcloud.com>
The header is just hidden if enabled is false.
But for the workspace this leads to continuous requests updating the content.

Signed-off-by: Max <max@nextcloud.com>
@mejo- mejo- force-pushed the backport/7501/stable31 branch from 2f3e74c to 8d03b62 Compare July 30, 2025 09:23
@mejo- mejo- merged commit 710847b into stable31 Jul 30, 2025
74 of 77 checks passed
@mejo- mejo- deleted the backport/7501/stable31 branch July 30, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants