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 |
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!
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).
~juther: 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. :-)
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.
Could you please have the "Quick No" button read "WARRANTS NO CHANGES" if that isn't too long? Thanks.
That was too long. I used "Exclude" instead. Implemented on DEV.
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!
Changes implemented. Don't forget to move tickets with changes back into the first column.
Thanks!
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.
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.
Wording change added.
Thanks, Bob! This looks great on DEV.
Verified on QA.
Verified on PROD.
Elapsed: 0:00:00.000730