Skip to content

Conversation

@rosh2525
Copy link

@rosh2525 rosh2525 commented Oct 7, 2025

This adds a portal property skin.hide_clinical_data_tab_study_view, which can hide the Clinical Data tab in the Study View page

Fixes cBioPortal/cbioportal#11653

Describe changes proposed in this pull request:

  • Added a new portal property, skin.hide_clinical_data_tab_study_view, to the IServerConfig interface.
  • This property is used in StudyViewPage.tsx to conditionally hide the "Clinical Data" tab.
  • Updated configuration loading to recognize window.portalProperties for local testing.

Checks

  • [ x] Has tests or has a separate issue that describes the types of test that should be created. If no test is included it should explicitly be mentioned in the PR why there is no test.

This is a minor visual change controlled by a configuration flag and does not require a dedicated test.

  • [ x] The commit log is comprehensible. It follows 7 rules of great commit messages. For most PRs a single commit should suffice, in some cases multiple topical commits can be useful. During review it is ok to see tiny commits (e.g. Fix reviewer comments), but right before the code gets merged to master or rc branch, any such commits should be squashed since they are useless to the other developers. Definitely avoid merge commits, use rebase instead.
  • [x ] Is this PR adding logic based on one or more clinical attributes? If yes, please make sure validation for this attribute is also present in the data validation / data loading layers (in backend repo) and documented in File-Formats Clinical data section!

Any screenshots or GIFs?

Screenshot 2025-10-05 at 3 32 32 PM Screenshot 2025-10-05 at 3 34 17 PM

Notify reviewers

Read our Pull request merging
policy
. It can help to figure out who worked on the
file before you. Please use git blame <filename> to determine that
and notify them either through slack or by assigning them as a reviewer on the PR

This adds a new portal property, skin.hide_clinical_data_tab_study_view, which allows to hide the Clinical Data tab in the Study View page. This fixes cBioPortal/cbioportal#11653
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE REQUEST] configuration to hide

1 participant