Skip to content

LIMS-1777: Fix download of autoprocessing attachments on sm beamlines#947

Merged
ndg63276 merged 1 commit intopre-release/2025-R3.2from
fix/LIMS-1777/fix-download-of-sm-attachments
Jun 11, 2025
Merged

LIMS-1777: Fix download of autoprocessing attachments on sm beamlines#947
ndg63276 merged 1 commit intopre-release/2025-R3.2from
fix/LIMS-1777/fix-download-of-sm-attachments

Conversation

@ndg63276
Copy link
Collaborator

@ndg63276 ndg63276 commented Jun 10, 2025

JIRA ticket: LIMS-1777

Summary:

Similar to #944, the attachments view of autoprocessing on 'sm' beamlines is broken as it thinks every visit has been purged.

Changes:

  • Pass the purged status down to DCAutoIntegrationView
  • Also, assume not purged if the purged status is undefined

To test:

  • Go to an I19 visit eg /dc/visit/cm40638-2, find a data collection with auto processing, click Logs & Files, check there are Download and View buttons, not just an iCat button.

@ndg63276 ndg63276 changed the base branch from master to pre-release/2025-R3.2 June 11, 2025 08:31
@ndg63276 ndg63276 marked this pull request as ready for review June 11, 2025 08:31
Copy link
Collaborator

@RichB-DLS RichB-DLS left a comment

Choose a reason for hiding this comment

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

Tested on local dev:

  • Go to an I19 visit eg /dc/visit/cm40638-2, find a data collection with auto processing, click Logs & Files, check there are Download and View buttons, not just an iCat button.

LGTM

@ndg63276 ndg63276 merged commit c7e3d51 into pre-release/2025-R3.2 Jun 11, 2025
2 checks passed
@ndg63276 ndg63276 deleted the fix/LIMS-1777/fix-download-of-sm-attachments branch June 17, 2025 09:55
ndg63276 added a commit that referenced this pull request Jun 24, 2025
* LIMS-1758: Fix slowness with plates with lots of subsamples (#941)

* LIMS-1758: Fix slowness with plates with lots of samples

* LIMS-1758: Remove unnecessary change

* LIMS-1758: Remove unnecessary whitespace

* LIMS-1758: Do large updates in chunks

* LIMS-1758: Use native JS

---------

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

* LIMS-1715: Clear sessionId from containers if a dewar transfer is requested (#939)

* LIMS-1715: Clear sessionId from containers if a dewar transfer is requested

* LIMS-1715: Clear sessionId from containers if a dewar transfer is requested

---------

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

* LIMS-1714: Populate Dewar.dewarRegistryId (#936)

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

* LIMS-1701: Make Xray Centring results an openable table (#935)

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

* LIMS-1642: Fix search when clicking x on search bars (#909)

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

* LIMS-1639: List most common screens at the top of the list (#928)

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

* LIMS-1777: Fix download of autoprocessing attachments on sm beamlines (#947)

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

---------

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