Skip to content

fix: don't throw when the list refresh after a failed upload fails - #4336

Merged
AlexAndBear merged 1 commit into
mainfrom
fix/upload-refresh-deleted-folder
Nov 9, 2026
Merged

AlexAndBear merged 1 commit into
mainfrom
fix/upload-refresh-deleted-folder

Conversation

@AlexAndBear

Copy link
Copy Markdown
Member

Description

Uploading into a folder that was deleted elsewhere no longer throws an uncaught error on every attempt and retry; the upload panel still shows the failure, and a folder whose files all failed still shows up in the list.

Related Issue

How Has This Been Tested?

  • test environment: unit tests (Vitest), browser tests (Playwright) against a local OpenCloud
  • test case 1: unit: a failed upload into a deleted folder resolves without throwing, the existing all-files-failed refresh still works
  • test case 2: browser: upload and retry into a deleted folder (desktop and Pixel 7, no page error), all files of a folder upload failing, normal file and folder uploads in the personal space and a project space incl. quota refresh

Types of changes

  • Bugfix
  • Enhancement (a change that doesn't break existing code or deployments)
  • Breaking change (a modification that affects current functionality)
  • Technical debt (addressing code that needs refactoring or improvements)
  • Tests (adding or improving tests)
  • Documentation (updates or additions to documentation)
  • Maintenance (like dependency updates or tooling adjustments)

@AlexAndBear
AlexAndBear merged commit 34e5776 into main Nov 9, 2026
29 checks passed
@AlexAndBear
AlexAndBear deleted the fix/upload-refresh-deleted-folder branch November 9, 2026 08:36
@openclouders openclouders mentioned this pull request Nov 9, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failed upload into a deleted folder throws an uncaught error

2 participants