Cbioportal: Add a optional warning to all pages

Created on 4 Apr 2018  路  14Comments  路  Source: cBioPortal/cbioportal

MSK users sometimes put internal urls in their publications. e.g.

http://ascopubs.org/doi/full/10.1200/JCO.2017.75.3384

image

We need to be able to add a warning and ask them not to do that.

  • [ ] add a new property in frontend or backend: study_view_warning
  • [ ] append the warning at the end of the description of a study
  • [ ] add a warning for all our private portal: "Please do NOT include the URL to this study or any other pages on this portal in your publication. If you need to make a study for your publication in the public cBioPortal, please contact [email protected]."
frontend good first issue study page

All 14 comments

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

this is still happening. But maybe the message I put above is a bit too long.

thoughts, @schultzn ?

How about just "Study URL is temporary - do NOT include in publications."

This applies to all studies in MSKCC/triage portals right? If so, maybe we can make it more general i.e. allow to add some blurb to the description of all studies in a portal. For example a skin.study_view.blurb. Similar to skin.blurb: https://docs.cbioportal.org/2.3-customization/portal.properties-reference#control-the-content-of-specific-sections.

@inodb sounds good. Yes, to all studies.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

we should do this

image

Thanks, @Leowisd. It looks good!

For the last one, could you add "transferred to the public cBioPortal"?

Is this message configurable? e.g. we should have a message in the private portal too.

@schultzn @n1zea144 comments?

Okay.

  • [x] Alert at top should read/appear as follows (notice wording change):
    image
  • [x] The UI/text in the modal should be changed as follows: (**= bold text)

Reverse the order of items so the link warning is first, and the guidelines second.

When adding a link to a cBioPortal cohort in a manuscript, **I will not link to the internal MSK portal (cbioportal.mskcc.org)**, but will instead link to this study on the public cBioPortal (cbioportal.org). Contact [email protected] with any questions about getting the data transferred to the public cBioPortal.

Got it

image

image

@Leowisd we only want to do this when
["triage-portal","mskcc-portal"].includes(AppConfig.serverConfig.app_name)

you may have to import AppConfig if it's not already imported on your page

Was this page helpful?
0 / 5 - 0 ratings