Skip to content

Conversation

@ObadaS
Copy link
Collaborator

@ObadaS ObadaS commented Sep 9, 2025

Original PR #1999 from @nicomy

@ mention of reviewers

@Didayolo @ObadaS @ihsaan-ullah

A brief description of the purpose of the changes contained in this PR.

This PR improves the Bulk Download functionality.

The preivous bulk download triggered multiple database queries, which was inefficient and error-prone. This PR refactors the process so that:

  • A single server-side query is executed.
  • The server generates download URLs and sends them to the client.
  • The client downloads files and builds a zip archive containing all submissions.

A progress bar UI try to give users feedback on the download progress.

Issues this PR resolves

  • Reduced database load by consolidating queries into one.
  • Avoided previous errors caused by multiple queries during bulk download.
  • Improved user experience with a visible progress indicator.

Related:

A checklist for hand testing

  • Verify that the updated view renders correctly and the layout is visually pleasing.
  • Trigger a bulk download on the test database and confirm that previous errors no longer occur.
  • Confirm that the progress bar appears only when the download starts and updates during the process.
  • Ensure that the zip file contains all expected submissions.

Any relevant files for testing

Previous Issue

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

@nicomy nicomy mentioned this pull request Sep 19, 2025
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants