Issue Number | 5355 |
---|---|
Summary | Summary QC reports not displaying comment or markup |
Created | 2024-12-23 14:40:48 |
Issue Type | Bug |
Submitted By | Osei-Poku, William (NIH/NCI) [C] |
Assigned To | Kline, Bob (NIH/NCI) [C] |
Status | Closed |
Resolved | 2024-12-23 19:03:26 |
Resolution | Fixed |
Path | /home/bkline/backups/jira/ocecdr/issue.484700 |
Crystal reported today that the Patient Summaries QC reports are not displaying comment or markup on PROD.
To Reproduce:
Retrieve a patient summary with Comment and Markup in XMetal (e.g. CDR0000799746)
Notice that it does have both Comment and markup (insertion and deletion)
Click on the PT RLSO report icon in the Summary toolbar
Scroll down to the Select Comment Types to be displayed section
Sellect All Comments
Click Submit
You should notice that there are no comments or markup in the report
Acceptance criteria:
Retrieve a patient summary with Comment and Markup in XMetal (e.g. CDR0000799746)
Notice that it does have both Comment and markup (insertion and deletion)
Click on the PT RLSO report icon in the Summary toolbar
Scroll down to the Select Comment Types to be displayed section
Sellect All Comments
Click Submit
You should notice that both markup and comments are displayed in the report
You've probably noticed this already, but it looks like the missing comment issue also affects HP summary reports as well.
Try it out on DEV.
I can see both the comments and markup on DEV. Thanks!
New tests have been added to the automated test suite to ensure that if we ever have a regression in which these bugs somehow reappear, we'll catch them. All of the new tests fail on QA (where the fixes won't be applied until Sanger is deployed there). All of them pass on DEV (where the fixes are installed).
Thanks Bob! Tagging ~vshields ~juther and ~duganal As it seems to me that we may want to fix this problem before Sanger since these features are needed for daily tasks.
This looks good on QA, but I would want Crystal to also confirm the fix before marking it as QA verified. She would be in the office tomorrow.
Verified on PROD. Thanks!
Elapsed: 0:00:00.001590