Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[dicom archive] Acquisition ID link not working correctly #9080

Closed
maximemulder opened this issue Feb 22, 2024 · 0 comments · Fixed by #9081
Closed

[dicom archive] Acquisition ID link not working correctly #9080

maximemulder opened this issue Feb 22, 2024 · 0 comments · Fixed by #9081
Assignees
Labels
26.0.0-bugs Issues that were raised during the release testing for 26.0.0 Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label)

Comments

@maximemulder
Copy link
Contributor

Describe the bug

The acquisition ID link in the dicom-archive/view-details page does not work correctly, as it does not filter the the mri_violations page according to the dicoms being viewed.

Example, the link on this page :
https://demo.loris.ca/dicom_archive/viewDetails/?tarchiveID=27
Currently sends the user to this page :
https://demo.loris.ca/mri_violations
Instead of this one :
https://demo.loris.ca/mri_violations?patientName=DCC090_587630_V1

@maximemulder maximemulder added Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) 26.0.0-bugs Issues that were raised during the release testing for 26.0.0 labels Feb 22, 2024
@maximemulder maximemulder self-assigned this Feb 22, 2024
driusan pushed a commit that referenced this issue Mar 11, 2024
Replace the obsolete link patches for the dicom-archive/view-details page with proper links.

Resolves #9080
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
26.0.0-bugs Issues that were raised during the release testing for 26.0.0 Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant