Skip to content

LIMS-1771: Don't display zero dose on grouped collections#951

Merged
ndg63276 merged 1 commit intopre-release/2025-R3.4from
fix/LIMS-1771/dont-display-zero-dose
Jul 8, 2025
Merged

LIMS-1771: Don't display zero dose on grouped collections#951
ndg63276 merged 1 commit intopre-release/2025-R3.4from
fix/LIMS-1771/dont-display-zero-dose

Conversation

@ndg63276
Copy link
Collaborator

JIRA ticket: LIMS-1771

Summary:

If a group of data collections has a total dose of zero, then that is displayed by the GUI. It should be hidden, the same as for individual data collections.

Changes:

  • Check dose above zero before displaying

To test:

  • Find a session with grouped data collections on i03, eg /dc/visit/mx35324-92/ty/fc, check the total dose is not displayed on the main page, nor is the individual dose shown when you click into the group
  • Find a session with grouped data collections on i04, eg /dc/visit/mx34598-83/ty/fc, check the total dose is displayed on the main page, and when you click into the group the dose for each individual collection is displayed

@ndg63276 ndg63276 changed the base branch from master to pre-release/2025-R3.4 July 8, 2025 10:29
@ndg63276 ndg63276 merged commit 4b63569 into pre-release/2025-R3.4 Jul 8, 2025
2 checks passed
@ndg63276 ndg63276 deleted the fix/LIMS-1771/dont-display-zero-dose branch July 8, 2025 10:30
ndg63276 added a commit that referenced this pull request Jul 22, 2025
* LIMS-1501: Allow re-running downstream processing (#948)

* LIMS-1501: Early work on re-running downstream processing

* LIMS-1501: Flesh out re-running downtream processing

* LIMS-1501: Remove unnecessary ProcessingJobParameters

* LIMS-1501: Enable fast_ep and some fast_dp

* LIMS-1501: Trigger zocalo directly

* LIMS-1501: Rename recipes for consistency

* LIMS-1501: Rename file more appropriately

---------

Co-authored-by: Mark Williams <mark.williams@diamond.ac.uk>

* LIMS-1771: Dont display zero dose on grouped collections (#951)

Co-authored-by: Mark Williams <mark.williams@diamond.ac.uk>

* LIMS-1688: Downstream processing wont display if filesystem not mounted (#953)

Co-authored-by: Mark Williams <mark.williams@diamond.ac.uk>

* LIMS-1739: Add button to say shipment is ready for scheduling (#945)

* LIMS-1739: Add button to say shipment is ready for scheduling

* LIMS-1739: Add button to say shipment is ready for scheduling

* LIMS-1739: Add contact instruction

* LIMS-1739: Dont show 'ready' button for industrial proposals

* Apply suggestions from code review

Co-authored-by: Guilherme Francisco <guilherme.de-freitas@diamond.ac.uk>

---------

Co-authored-by: Mark Williams <mark.williams@diamond.ac.uk>
Co-authored-by: Guilherme Francisco <guilherme.de-freitas@diamond.ac.uk>

---------

Co-authored-by: Mark Williams <mark.williams@diamond.ac.uk>
Co-authored-by: Guilherme Francisco <guilherme.de-freitas@diamond.ac.uk>
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