Skip to content

LIMS-1770: Fix download of attachments on saxs and xpdf beamlines#944

Merged
ndg63276 merged 1 commit intopre-release/2025-R3.1from
fix/LIMS-1770/cannot-download-attachments-from-saxs-or-xpdf
Jun 5, 2025
Merged

LIMS-1770: Fix download of attachments on saxs and xpdf beamlines#944
ndg63276 merged 1 commit intopre-release/2025-R3.1from
fix/LIMS-1770/cannot-download-attachments-from-saxs-or-xpdf

Conversation

@ndg63276
Copy link
Collaborator

@ndg63276 ndg63276 commented Jun 5, 2025

JIRA ticket: LIMS-1770

Summary:

Since #921 was deployed, visits on B18 and B21 don't allow you to download attachments, only giving you a link to iCat instead.

Changes:

  • Pass the dcPurgedProcessedData variable down into the downstream view from SAXS dc.js and XPDF dc.js

To test:

  • Go to a B18 visit (eg /dc/visit/cm40637-2), click on an AutoProcessing line, then click Logs & Files. Check the Download and View buttons appear, not just a link to iCat
  • Repeat for a B21 visit (eg /dc/visit/cm40642-2)

@ndg63276 ndg63276 marked this pull request as ready for review June 5, 2025 14:42
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.

Both tests worked on staging-1. LGTM

@ndg63276 ndg63276 changed the base branch from master to pre-release/2025-R3.1 June 5, 2025 14:58
@ndg63276 ndg63276 merged commit 921a075 into pre-release/2025-R3.1 Jun 5, 2025
2 checks passed
@ndg63276 ndg63276 deleted the fix/LIMS-1770/cannot-download-attachments-from-saxs-or-xpdf branch June 5, 2025 14:59
ndg63276 added a commit that referenced this pull request Jun 10, 2025
* LIMS-1447: Don't allow shipping of red samples (#929)

* LIMS-1664: Improvements to registered dewar and container pages (#927)

* LIMS-1636: Make LDAP server ID field configurable (#934)

* LIMS-1699: Show confirmation dialog if a user creates a plate that is not full (#924)

* LIMS-1713: Allow sm beamlines to use mx Create Container page (#932)

* LIMS-1770: Fix download of attachments on saxs and xpdf beamlines (#944)
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