EBMS Tickets

Issue Number 438
Summary [Literature] Add "Quick No" Button Beside "Review" Button with Pop-up Box
Created 2017-05-31 17:40:58
Issue Type Improvement
Submitted By Juthe, Robin (NIH/NCI) [E]
Assigned To Kline, Bob (NIH/NCI) [C]
Status Closed
Resolved 2017-06-16 14:00:19
Resolution Fixed
Path /home/bkline/backups/jira/oceebms/issue.209336
Description

Some Board members have expressed a desire to be able to quickly take care of the "no's" in their literature packets without having to go to another page. We thought we might address this by adding a "Quick No" (we need a better name) button beside the "Review" button for each article. If a Board member clicked on the "Quick No" button, a pop-up box would be presented from which the member would be required to select one or more exclusion reason and optionally provide a comment, before submitting the decision. Once a decision has been submitted, the "Quick No" and "Review" buttons would be replaced by the standard "reviewed MM/DD/YYYY" text that we display now once an article has been reviewed. We are not requesting any changes to the "Review" page - a member could still record "no changes" on that page.

I will think on the "Quick No" button name, but it would be helpful to get a story point estimate to see if this is worth pursuing at this time. Let me know if you need more info. Thanks!

Comment entered 2017-06-01 13:29:31 by Kline, Bob (NIH/NCI) [C]

The story point estimate would be 8 if we have the "No" button bring up a page with a form and then just re-draw the packet page when the user submits the form (rather than having a popup manipulate parts of an existing page with ajax).

Comment entered 2017-06-07 16:21:48 by Kline, Bob (NIH/NCI) [C]

: I'm getting closer to this ticket, so I wanted to give you an opportunity to consider whether you would prefer to have me implement the safer and less expensive approach to this request, as mentioned in my previous comment. :-)

Comment entered 2017-06-07 16:27:36 by Juthe, Robin (NIH/NCI) [E]

Hi Bob. We discussed this issue in our meeting today and our preference is still for the pop-up box. If you think this is too risky to implement, let me know. Thanks.

Comment entered 2017-06-08 13:56:53 by Juthe, Robin (NIH/NCI) [E]

Could you please have the "Quick No" button read "WARRANTS NO CHANGES" if that isn't too long? Thanks.

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

That was too long. I used "Exclude" instead. Implemented on DEV.

Comment entered 2017-06-14 16:40:39 by Juthe, Robin (NIH/NCI) [E]

This looks great! I think our Board members are really going to appreciate this update.

A few small requests:

1. Please change "EXCLUDE" to "REJECT". We decided that was a little bit more meaningful. Semantics... 🙂

2. Please change the heading of the pop-up box from "Excluding article PMID 12345678" to "Rejecting article Smith JAMA 2015 PMID 12345678" where Smith is the first author's last name, JAMA is the journal abbreviation, and 2015 is the year. You may need to establish a character cut-off in case the abbreviated journal title is really long - not sure.

Thanks!

Comment entered 2017-06-14 18:12:04 by Kline, Bob (NIH/NCI) [C]

Changes implemented. Don't forget to move tickets with changes back into the first column.

Thanks!

Comment entered 2017-06-16 10:57:40 by Juthe, Robin (NIH/NCI) [E]

I think we need to add a statement about the "reject" button in the instructions at the top of the page. Should we handle that as part of this issue or would you like a separate ticket? I'm thinking we could add the following sentence to #2: "Use the REJECT button to quickly reject any articles that warrant no changes to the summary." Thanks.

Comment entered 2017-06-16 12:07:43 by Kline, Bob (NIH/NCI) [C]

I can make the change as part of this ticket, but not right now. CBIIT has blown away access to my home directory on the DEV EBMS server, so I can't get to my SVN sandbox (this happened once before, about a week ago). I'll let you know when they've fixed the problem.

Comment entered 2017-06-16 14:00:19 by Kline, Bob (NIH/NCI) [C]

Wording change added.

Comment entered 2017-06-16 14:06:59 by Juthe, Robin (NIH/NCI) [E]

Thanks, Bob! This looks great on DEV.

Comment entered 2017-06-28 17:15:26 by Juthe, Robin (NIH/NCI) [E]

Verified on QA.

Comment entered 2017-08-22 17:50:05 by Juthe, Robin (NIH/NCI) [E]

Verified on PROD.

Elapsed: 0:00:00.000730