Issue Number | 340 |
---|---|
Summary | [Import] Add "Published" as a placement level for fast track citations |
Created | 2015-11-12 11:56:00 |
Issue Type | Improvement |
Submitted By | trivedim |
Assigned To | Kline, Bob (NIH/NCI) [C] |
Status | Closed |
Resolved | 2015-12-04 09:11:03 |
Resolution | Fixed |
Path | /home/bkline/backups/jira/oceebms/issue.173606 |
At present the default placement level is blank and if we use the
arrow for pull down option, it displays other options.
We would like to add “Passed Medical Librarian’s Review” as a placement
level when we use the TAG “Core Journals Search”. I don’t think we need
other placement levels for importing core journals.
Also we would like the citations to be automatically “Published”.
Minaxi, thanks for entering this ticket. It looks like the first part of your request is already being handled on QA without needing to select a decision from the placement level picklist. Citations that are imported with the "core journals search" tag automatically receive a "passed medical librarian review" decision. For example, see EBMS ID: 404772 on QA. I just imported this with the "core journals" box checked and this applied both the tag and the passed medical librarian review decision. I believe the only thing that isn't happening is that the citation is automatically published (I just published this citation manually).
Yes that is correct. It would save us a lot of time if it is
automatically published.
Also it would help to add “Passed Medical Librarian’s Review” as a
placement level.
Thanks, MInaxi. Would you use the "passed medical librarian review" placement level for imports in other situations? It's fine to add it if this would be helpful for other situations, but if it's just for these core journal imports, then I'm not sure if it's necessary since the system is already applying that placement level for these citations (based on applying the tag) without you needing to select it.
Added Cynthia to this issue. Cynthia, since Minaxi is out, could you please clarify what is needed for this ticket? (please see my question in the previous comment) Thanks!
Holding off on estimating this ticket until the requirements are clarified.
Yes I think we could use this "Passed Medical Librarian review" placement level in other ways in additiona to the core journal search result. I think it could also be used for special search requests that Minaxi and I review before we import. And therefore save us the step of having to re-review them after import just to get them bumped up to the NCI Reviewers queue.
Enhancement implemented on DEV.
Bob, could you please use sentence case - "Passed medical librarian review" to match (most of) the other labels in the drop-down list? If you're able to do this at the same time, please lowercase the "a" in "On Agenda" in that list, too. If not, we can handle that another time. Thanks.
Capitalization adjusted as requested.
Thanks! Sorry, but "Board" should remain capitalized...we always capitalize it when we're referring to the Boards.
Capitalization tweaked again (as requested).
This worked the first time but then the option disappeared. The pull down menu disappeared from the import screen. I clicked on several other things (publishing and searching) and came back to the import page and the placement level pull down menu was still gone. I had to exit and log back in to get it to reappear.
I can also get it to reapear when I select a board.
If this is going to appear selectivly, shouldn't it only appear when the
core journals or fast track boxes are checked?
The timing of the appearance of the picklist for placement hasn't changed (it's behaving the same way in production). To have this modified, please file another ticket.
I'll file a separate ticket for the behavior of the placement level menu. It seems to be working okay for me, but that may be because I have a default Board that is always selected when I go to that page. Or perhaps I'm just used to it's sporadic appearance! I'll look into a bit more and put in an issue. I agree that it should only appear when relevant, but I think it should just be added when the "fast track" option is selected since what happens with the core journal tag is a bit different. Citations tagged with "core journal" both pass the medical librarian review AND get published for the imported topic.
Cynthia, did you try importing any citations with the new placement level? I have tried that and it seems to work fine.
Bob, did you say this will work for importing a string of PMIDs but NOT batch import files? Just wanted to capture that in the ticket if I have that right.
Thank you both,
Robin
I'm pretty sure it will work both with a string of PMIDs and with batch import files. Let me know if it's not behaving that way.
OK, sorry to confuse things - thank you for clarifying. It will be good for Cynthia to test the batch part then since I am not accustomed to working with batch import files.
I imported a batch of 9 childhood liver citations from a file. I checked the box for core journals and choose passed med lib rev. They show as passing my review but are awaiting publication. The citations did not get automatically published. See 404782.
I just imported 2 citations using the pmid in a string. I checked the box for core journals and selected passed med lib rev. They also show as passing my review but are also awaiting publication. These citations did not get automatically published either. See 404790
Looking back at the history for both tickets, it appears the publishing part has not been implemented. Shall I file a separate ticket for that Bob, or do you want to handle it as part of this ticket? It's mentioned in Minaxi's original description, but then I think we lost it when clarifying the requirements (and I forgot it hadn't been implemented already!).
Cynthia, do you want the citations to be automatically published in both cases (i.e., when selecting either "core journals" or "passed medical librarian review"), or just when you select the core journal checkbox?
Yes, both. Core journals because these are citations that have been selected and reviewed prior to import so they are ready to immediately get published. And any citation that we want to automatically pass my review, can get published. This just saves us a few extra steps.
I see. So, really the placement level for OCEEBMS-281 should have been "Published," not "PassedInitReview"?
Changed placement level of articles imported with the core journals flag to "Published" (confirmed verbally with Robin). Changed on both DEV and QA (and in Subversion).
Thanks, Bob. I just tested that on QA with a single PMID and it worked well! Cynthia, please see what you think and try a batch also.
I think Cynthia is saying above (referring to her comment yesterday at 4:16) that she would like the placement level selection (this ticket) to also result in the citations being published. Given that change, I think we should consider rewording the placement level to "Published" since it's no longer just passing her review. Would you like a separate ticket for that?
Tell you what: you change the title of this ticket, and I'll make the change to the software. :-)
Done. Thank you, Bob!!!
Replaced "PassedInitReview" with "Published" in placement level picklist on DEV and QA.
This looks good. I have tested with batches of citations from a text file and with strings of pmids as well as single pmid. placement level works, core journals check box does what it is supposed to. This feature will save Minaxi and I alot of extra steps!
Verified on QA.
This is in response to comments by Cynthia and Robin regrding the
placement level disappearing after importing one citation. I tested it
today and experinced the same. If I have 10 core journals citations to
import for PediatricBoard, it allows me to select placement level for
the first cittaion. After that I have to select another Board and then
come back to Pediatric Board to import next citation. So Basically the
placement level should stay at all the time as long as the Core Journals
tag is selected.
Robin, would you like me to open the new ticket for the issue?
Thanks for testing this Minaxi! I've added a separate ticket for the problem with the disappearing placement level - see OCEEBMS-351. We hope to tackle this is iteration 3 of this release.
Thanks, Robin!
Hope to see this working soon. It will save a lot of time for me.
Verified on PROD.
Elapsed: 0:00:00.000660