CDR Tickets

Issue Number 4217
Summary [Summaries] Type of Change Report - Changes to Interface
Created 2017-01-17 14:43:38
Issue Type Improvement
Submitted By Juthe, Robin (NIH/NCI) [E]
Assigned To Kline, Bob (NIH/NCI) [C]
Status Closed
Resolved 2017-01-31 23:22:15
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.201526
Description

The interface for the Type of Change report is confusing. It isn't clear that the Format, Audience, For Specific Summaries, and For Multiple Summaries options are applicable to both versions of the report - i.e., the most recent Type of Change and the Type of Change History.

I'll post my suggestions for reformatting this page.

Comment entered 2017-01-23 11:07:37 by Kline, Bob (NIH/NCI) [C]

I have created a mockup of a new form. I'll plug in the code to generate the report after we've worked in any tweaks the form needs. Please take a look.

https://cdr-dev.cancer.gov/cgi-bin/cdr/stcr.py?Session=guest

Comment entered 2017-01-23 21:06:38 by Kline, Bob (NIH/NCI) [C]

The stcr.py script now generates the reports. I left it under the temporary name to make it easier for you to compare results between the old and new versions of the code.

Comment entered 2017-01-26 14:13:41 by Osei-Poku, William (NIH/NCI) [C]

In the historical report, please flip the order of the changes so that the most recent change is at the top for each summary.

As a reminder, please also fix the column names.

Comment entered 2017-01-27 06:26:12 by Kline, Bob (NIH/NCI) [C]

Robin is reviewing this on DEV.

Comment entered 2017-01-31 23:22:07 by Kline, Bob (NIH/NCI) [C]

Both changes made on DEV.

Comment entered 2017-02-02 12:51:06 by Juthe, Robin (NIH/NCI) [E]

Bob, could you please hide the date fields when the "current" version of the report is selected on the default report interface? Thanks.

Comment entered 2017-02-02 13:15:42 by Kline, Bob (NIH/NCI) [C]

Done.

Comment entered 2017-02-10 17:00:37 by Juthe, Robin (NIH/NCI) [E]

I'm seeing some discrepancies beyond what I'm expecting to see between the two reports.

For instance, when I run the "current (most recent change)" version of the new report and the "basic" version of the old report, I'm seeing changes from 2013 on the new report when the old report is displaying more recent changes from 2015 for the same categories.

Here's a concrete example:

1. Run the "current" or "basic" report (both old and new versions) for all genetics summaries, all change types and including comments.
2. For the Genetics of Breast/Gyn Cancers summary, the new report shows the most recent comprehensive revision change as being from 2014-12-03, while the old report shows the most recent change (which happens to be a comprehensive revision) dated 2015-11-11. I would expect the new report to show me the change from 2015 in the comprehensive revision column.
3. For the Genetics of Colorectal Cancer summary, a similar thing is happening in the Major Change column - the new report shows a change from 2013 while the old report displays a change from 2015.
4. The same is true for the Endocrine, Kidney, Prostate, and Skin summaries in either the Comprehensive revision or Major change columns.

Comment entered 2017-02-10 17:53:21 by Kline, Bob (NIH/NCI) [C]

Ah! I see what happened. When we reversed the order in which the changes are stored for the change request received on the 26th, that flipped the logic for the other flavor of the report. Try it again please.

Comment entered 2017-02-13 13:00:49 by Juthe, Robin (NIH/NCI) [E]

This looks good although do you still have the link to the old report for comparison purposes?

Comment entered 2017-02-13 13:16:40 by Kline, Bob (NIH/NCI) [C]

I added a link back to the old version on the Board Manager menus (but not the general summary reports menu) on DEV.

Comment entered 2017-02-13 13:32:05 by Juthe, Robin (NIH/NCI) [E]

Thanks, but unfortunately the old report isn't working. I'm getting a "CGI parameter tampering detected" error. If it isn't easy to fix, don't worry about it and I can just proceed with testing the new report.

Comment entered 2017-02-13 13:53:43 by Kline, Bob (NIH/NCI) [C]

Sorry about that. Give it another shot.

Comment entered 2017-02-13 13:55:16 by Juthe, Robin (NIH/NCI) [E]

All better, thanks!

Comment entered 2017-02-13 14:03:55 by Juthe, Robin (NIH/NCI) [E]

The "historical" version of the new report is ignoring the selected change types. It should display all changes of the selected type(s) for the selected date range. Thanks.

Comment entered 2017-02-13 14:38:53 by Kline, Bob (NIH/NCI) [C]

Evidently I thought that's how the historical report should work, given the wording on the user interface. I've changed the behavior, and when you've confirmed that it's working the way you want, I'll change the wording on the report request form too.

Comment entered 2017-02-16 17:31:16 by Juthe, Robin (NIH/NCI) [E]

Hi , this looks good to me. I'm considering it verified on DEV. You can remove the link to the old version of the report. Thank you!

Comment entered 2017-02-24 12:06:43 by Juthe, Robin (NIH/NCI) [E]

The formatting of this report interface is a little wacky on QA.

Comment entered 2017-02-24 16:45:28 by Kline, Bob (NIH/NCI) [C]

Could you elaborate on "wacky"? I'm not seeing wacky. :-) Maybe you need to refresh your browser's cache?

Comment entered 2017-02-24 16:49:06 by Juthe, Robin (NIH/NCI) [E]

Can we no longer add a screenshot in JIRA? I will send you one. Maybe it's just me - I will try updating my cache.

Comment entered 2017-03-03 13:20:55 by Juthe, Robin (NIH/NCI) [E]

We confirmed this problem was only on my computer.

Comment entered 2017-03-03 13:21:04 by Juthe, Robin (NIH/NCI) [E]

Verified on QA.

Comment entered 2017-03-10 17:07:28 by Juthe, Robin (NIH/NCI) [E]

Bob, the "old" Type of Change report is still on Stage. Should it be?

Comment entered 2017-03-13 09:40:07 by Kline, Bob (NIH/NCI) [C]

JIRA's broken again, unable to show previous comments, so I can't just "reply" to your question about the link to the old interface, Robin. Looks like we missed that on QA. It'll be fixed on PROD.

Elapsed: 0:00:00.001362