Skip to content

fix: swap swapped NGS/proteomics raw data download filenames - #1485

Merged
sven1103 merged 2 commits into
developmentfrom
fix/1444-swapped-raw-data-filenames
Jul 28, 2026
Merged

fix: swap swapped NGS/proteomics raw data download filenames#1485
sven1103 merged 2 commits into
developmentfrom
fix/1444-swapped-raw-data-filenames

Conversation

@sven1103

Copy link
Copy Markdown
Contributor

Summary

Fixes #1444 — The url.txt download filename was hardcoded with the wrong suffix for NGS and proteomics tabs.

Problem

When downloading url.txt from the raw data view, the file names were swapped:

  • NGS tab → <date>_<project>_proteomics_measurement_dataset_locations.txt
  • PxP tab → <date>_<project>_ngs_measurement_dataset_locations.txt

Fix

Swapped the filenames so they match the selected tab:

  • NGS tab → <date>_<project>_ngs_measurement_dataset_locations.txt
  • PxP tab → <date>_<project>_proteomics_measurement_dataset_locations.txt

Changes

  • datamanager-app/src/main/java/life/qbic/datamanager/views/projects/project/rawdata/RawDataDetailsComponent.java

Type

Bug fix — filename swap in RawDataDetailsComponent.addNgsTab() and addPxpTab().

- NGS tab now produces <date>_<project>_ngs_measurement_dataset_locations.txt
- PxP tab now produces <date>_<project>_proteomics_measurement_dataset_locations.txt

Fixes #1444
@sven1103
sven1103 requested a review from a team as a code owner July 24, 2026 11:34
@github-actions github-actions Bot added the fix label Jul 24, 2026
@sven1103 sven1103 linked an issue Jul 24, 2026 that may be closed by this pull request
1 task
@sonarqubecloud

Copy link
Copy Markdown

@KochTobi KochTobi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sven1103
sven1103 merged commit a7ece6b into development Jul 28, 2026
5 of 6 checks passed
@sven1103
sven1103 deleted the fix/1444-swapped-raw-data-filenames branch July 28, 2026 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] url.txt file name

2 participants