Add progress tracking to file upload #189
Conversation
🦋 Changeset detectedLatest commit: 0c61015 The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
||
| if (files.length) { | ||
| // Upload all files that came from the object | ||
| const uris = await this.uploader.uploadBatch(files); |
There was a problem hiding this comment.
oof.. make sure that all other calls pass the options properly
There was a problem hiding this comment.
This one is tricky - the way we used to do it is only pass through options (and onProgress) through to one of the calls on batchUpload because otherwise the progress would reset in the middle (since we upload files first, then the top-level object). So we may want to only pass through onProgress to the file upload.
Can keep this as a fix for now though
|
/release-pr |
|
/release-pr |
No description provided.