Skip to content

Exclude hold delivery in the automation.#5158

Open
RasmusBurge-CG wants to merge 5 commits into
masterfrom
exclude-hold-delivery
Open

Exclude hold delivery in the automation.#5158
RasmusBurge-CG wants to merge 5 commits into
masterfrom
exclude-hold-delivery

Conversation

@RasmusBurge-CG

@RasmusBurge-CG RasmusBurge-CG commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Description

To be merged with:
https://github.com/Clinical-Genomics/cigrid-ui/pull/856
Clinical-Genomics/trailblazer#536

Make the queries to Trailblazer that are used in the DeliverService filter out the analyses that are manually set to be held out of automatic delivery

Added

  • Term &holdDelivery=false when querying for analyses in trailblazer with API methods:
    • get_analyses_to_deliver_for_order
    • get_analyses_to_deliver_for_case
    • get_all_analyses_to_deliver

How to prepare for test

  • Ssh to relevant server (depending on type of change)
  • Use stage: us
  • Paxa the environment: paxa
  • Install on stage (example for Hasta):
    bash /home/proj/production/servers/resources/hasta.scilifelab.se/update-tool-stage.sh -e S_cg -t cg -b exclude-hold-delivery -a

How to test

  • Deploy Add Hold Delivery functionality trailblazer#536 in trailblazer stage
  • Make a few analyses visible in Trailblazer: From the same order, different case, one of them set to hold for automatic delivery
  • Call
    • cg deliver case <case_internal_id> -s SD with the case held for automatic delivery
    • cg deliver order -t <ticket_id> -s SD
    • cg deliver all-available

Expected test outcome

  • Check that in any case, the case set as hold for automatic delivery is delivered

Review

  • Tests executed by
  • "Merge and deploy" approved by SD
    Thanks for filling in who performed the code review and the test!

This version is a

  • MAJOR - when you make incompatible API changes
  • MINOR - when you add functionality in a backwards compatible manner
  • PATCH - when you make backwards compatible bug fixes or documentation/instructions

Implementation Plan

  • Document in ...
  • Deploy this branch on ...
  • Inform to ...

@RasmusBurge-CG RasmusBurge-CG requested a review from a team as a code owner June 25, 2026 08:54
@diitaz93 diitaz93 changed the title Exclude holder delivery in the automation. Exclude hold delivery in the automation. Jun 25, 2026
@diitaz93 diitaz93 marked this pull request as draft June 25, 2026 11:04
@diitaz93 diitaz93 marked this pull request as ready for review June 25, 2026 13:47
@sonarqubecloud

Copy link
Copy Markdown

@diitaz93 diitaz93 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.

💯

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.

2 participants