Skip to content

feat(upload): merge upload related API from @nextcloud/upload#1509

Draft
susnux wants to merge 14 commits intomainfrom
feat/merge-uploader
Draft

feat(upload): merge upload related API from @nextcloud/upload#1509
susnux wants to merge 14 commits intomainfrom
feat/merge-uploader

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Feb 19, 2026

As it was decided to move UI to the vue library it is easier to move the pure API to this library.
Reducing libraries to maintain and also it fits nicely into this libraries scope.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
…and folders

This also refactors the uploader to support custom events
allowing apps to register event listeners for reactivity.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
This allow to better visualize ongoing uploads in an UI

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux added type: enhancement 🚀 New feature or request 2. developing 2️⃣ Work in progress labels Feb 19, 2026
@codecov
Copy link

codecov bot commented Feb 19, 2026

Codecov Report

❌ Patch coverage is 36.81592% with 254 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.47%. Comparing base (9baccb0) to head (2ba1041).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
lib/upload/uploader/UploadFileTree.ts 0.00% 73 Missing ⚠️
lib/upload/uploader/Uploader.ts 15.66% 69 Missing and 1 partial ⚠️
lib/upload/utils/fileTree.ts 0.00% 38 Missing ⚠️
lib/upload/uploader/UploadFile.ts 60.52% 26 Missing and 4 partials ⚠️
lib/upload/utils/upload.ts 12.12% 29 Missing ⚠️
lib/upload/utils/config.ts 38.46% 6 Missing and 2 partials ⚠️
lib/upload/utils/filesystem.ts 0.00% 3 Missing ⚠️
lib/upload/uploader/Upload.ts 60.00% 2 Missing ⚠️
lib/upload/uploader/Eta.ts 98.41% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1509       +/-   ##
===========================================
- Coverage   86.14%   67.47%   -18.67%     
===========================================
  Files          30       43       +13     
  Lines         664     1070      +406     
  Branches      194      269       +75     
===========================================
+ Hits          572      722      +150     
- Misses         80      326      +246     
- Partials       12       22       +10     

☔ 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing 2️⃣ Work in progress type: enhancement 🚀 New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant