Skip to content

LIMS-2108: Add more pipelines to the summary page#1046

Merged
ndg63276 merged 1 commit intopre-release/2026-R2.1from
improvement/LIMS-2108/add-more-pipelines-to-summary-page
Mar 24, 2026
Merged

LIMS-2108: Add more pipelines to the summary page#1046
ndg63276 merged 1 commit intopre-release/2026-R2.1from
improvement/LIMS-2108/add-more-pipelines-to-summary-page

Conversation

@ndg63276
Copy link
Copy Markdown
Collaborator

@ndg63276 ndg63276 commented Mar 9, 2026

JIRA ticket: LIMS-2108

Summary:

The summary page currently has hard-coded pipelines, we should get them from the ProcessingPipelines table. That way we get the Multi pipelines, and any new ones added in future.

Changes:

  • Get processing pipelines from the DB
  • Allow fetching of pipelines with different statuses
  • Use the pipeline id to fetch matching processing runs

To test:

  • Go to a visit with multiple successful processing runs, then click the Summary button to go to eg /dc/summary/visit/mx23694-156
  • Check the list of pipelines now includes multi- pipelines
  • Check selecting a pipeline only shows results from that pipeline (results from multi-* and STARANISO may not appear yet until Data Analysis start tagging them)
  • Go to a shipment, create a puck, check you can still choose from 3 pipelines for priority processing

@ndg63276 ndg63276 marked this pull request as ready for review March 10, 2026 13:09
@gfrn
Copy link
Copy Markdown
Collaborator

gfrn commented Mar 16, 2026

Tested post DB inserts, LGTM

@ndg63276 ndg63276 changed the base branch from master to pre-release/2026-R2.1 March 24, 2026 10:17
@ndg63276 ndg63276 merged commit a582fc4 into pre-release/2026-R2.1 Mar 24, 2026
2 checks passed
@ndg63276 ndg63276 deleted the improvement/LIMS-2108/add-more-pipelines-to-summary-page branch March 24, 2026 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants