Skip to content

Conversation

@labkey-klum
Copy link
Contributor

@labkey-klum labkey-klum commented Dec 5, 2025

Rationale

This PR removes some obsolete functionality in the study module:

  • Stop saving the uploaded data to files during a dataset bulk import.
  • Don't render a download link in the import history grid view (the filePath field in the table is still retained)

Tasks 📍

  • Manual Testing
    • Verified that importing files into datasets still adds rows to the upload history
    • Verified that no "download data file" links or column appear
    • Verified that filepath column is NULL for new entries
    • Verified that uploaded files don't appear in the file root
    • Interestingly, I had no existing UploadLog rows in any of my databases. This table must not be populated when importing from a folder archive, which seems odd.
  • Verify Fix

@labkey-klum labkey-klum merged commit e8c888f into release25.11-SNAPSHOT Dec 5, 2025
9 of 11 checks passed
@labkey-klum labkey-klum deleted the 25.11_fb_issue_53922_part2 branch December 5, 2025 23:12
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