Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[i858] - Enables chunked uploads for improved file upload reliability #982

Merged
merged 13 commits into from
Oct 30, 2024

Conversation

ShanaLMoore
Copy link
Contributor

@ShanaLMoore ShanaLMoore commented Oct 25, 2024

Issue:

Chunked uploads allow large files to be uploaded in smaller parts, addressing common issues with file size limits, network interruptions, and server resource management.

This change supports smoother, more reliable uploads for large files in Bulkrax.

When a user uploads, multiple POST requests can be seen in the network tab because the uploads gets split into chunks. It's then handed to Hyrax controller to handle it on the server side.

image

DEMO:

Zight Recording 2024-10-29 at 09 49 23 AM

CSV

Screenshot 2024-10-28 at 13-59-46 New Importer __ Hyku

if user tries to upload more than 1 file:

image

BAGIT

Screenshot 2024-10-28 at 13-59-32 New Importer __ Hyku

XML

Screenshot 2024-10-28 at 13-27-59 New Importer __ Hyku

… and efficiency

Issue:
- scientist-softserv/adventist_knapsack#858

Chunked uploads allow large files to be uploaded in smaller parts, addressing common issues with file size limits, network interruptions, and server resource management.

This change supports smoother, more reliable uploads for large files in Bulkrax.
@ShanaLMoore ShanaLMoore added the minor-ver for release notes label Oct 28, 2024
ShanaLMoore and others added 6 commits October 29, 2024 12:28
… initialization

- Moved Bulkrax file upload logic from bulkrax.js to importers.js to consolidate related logic
- Added conditional check to initialize uploader only if `hyraxUploader` is defined as a function
- Ensured toggle of 'required' attribute based on uploaded file presence for import form consistency
@ShanaLMoore ShanaLMoore marked this pull request as ready for review October 29, 2024 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor-ver for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants