Issue Number | 5047 |
---|---|
Summary | Modify summaries schema to include “On This Page” for a SVPC summary |
Created | 2021-10-15 14:57:49 |
Issue Type | New Feature |
Submitted By | Osei-Poku, William (NIH/NCI) [C] |
Assigned To | Kline, Bob (NIH/NCI) [C] |
Status | Closed |
Resolved | 2021-10-25 12:11:31 |
Resolution | Fixed |
Path | /home/bkline/backups/jira/ocecdr/issue.300672 |
As a CDR patient summary editor, I want to be able to mark a SVPC summary so that the “On This Page” section is displayed on Cancer.gov.
Currently, top level section titles are included in the “On This Page” section of summaries on Cancer.gov. However, for SVPC summaries, they will not be displayed by default. But there will be cases where the “On This Page” section will be displayed for a SVPC summary. We want to be able to mark a SVPC summary (at the page level) to have “On This Page” displayed on Cance.gov.
Given I am a CDR patient summary editor
And I have permissions to create summary documents
And I have permissions to edit summary documents
When I create or edit a SVPC summary
Then I want to be able to mark it as needing the “On This Page” section on Cancer.gov
I would like to get some clarification on this ticket.
In the beginning we were talking about not displaying the On This Page section. Later on I heard, as the first scenario of this ticket describes, that the On This Page section should be optional and we need to be able to turn its display on or off. The second scenario, however, is a bit confusing to me as the description is talking about the On This Page section to be optional while the heading indicates that the section title should be optional. It sounds that we should be able to optionally display the On This Page section and - if the section is displayed - it should be optional which of the section titles should be included in the On This Page section. Is this correct?
Also, I'm assuming that nothing should be displayed if an On This Page section is marked to be displayed but all of the section titles are marked not to be displayed
~oseipokuw - Could you
please identify the part of the schema you would like to have modified?
I am unable to find the word "page" (any case) anywhere on the
SummarySchema
or SummaryCommon
documents.
Hi ~bkline I think this should be an attribute on the <Summary> element like the new SVPC attribute. For example, we can name the attribute "SVPCOnThisPage" with a blank default value (Indicating that "On This Page" should be displayed on Cancer.gov) and with the option to select "No" indicating "On This Page" should not be displayed on Cancergov. This will be a flag for the vendor filters. I will create another ticket to modify the vendor filters.
~bkline, the "On This Page" section gets created by JavaScript based on header elements/KeyPoints of the section.
The attribute ~oseipokuw suggests for the Summary element will work to turn the "On This Page" on or off. However, it wouldn't allow individual section titles to be suppressed (see my earlier comment/question on this ticket).
Right.
~oseipokuw: please see ~volker's request for clarification above.
My understanding was that the default for SVPC documents would be not to display the On This Page. Wouldn't it make sense to use the attribute only to overwrite the default (SVPCOnThisPage="Yes")?
My understanding was that the default for SVPC documents would be not to display the On This Page. Wouldn't it make sense to use the attribute only to overwrite the default (SVPCOnThisPage="Yes")?
Yes, that would make sense. I thought it was the other way around.
From the status meeting: this is a document-specific switch, controlling whether to generate the On This Page block for the summary. There will not be a mechanism for controlling which sections are represented in the block.
As a side note, Key Points are going away for these summaries.
Installed on DEV. ~volker I was able to get back into Remote Apps from my own MacBook (apparently it like going to a speakeasy — you have to whisper a secret passphrase through a keyhole and hope that the cops aren't around and there's an empty seat available inside), run the git client from there and post the updated schema using the browser interface.
~oseipokuw, it might be less confusing if at least the title of the issue were modified to reflect the most recent discussion you and Volker had in this issue.
Verified on DEV. Thanks!
We decided to change the name of the attribute to
SuppressOnThisPageSection
. The schema has been modified
accordingly on DEV.
Verified on QA and PROD. Thanks!
Elapsed: 0:00:00.001764