Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add configuration to disable study tags #10637

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

alisman
Copy link
Contributor

@alisman alisman commented Feb 15, 2024

Configuration property allows portal maintainers to disable study tag functionality, which is enabled by default. Study tags are miscellaneous meta data associated with individual cancer studies.

## Study Tag functionality
#enable_study_tags=true|false

# EOL - Do not delete the following lines
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you might have accidentally deleted the line after this. If you add it back then the builds should work.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@alisman alisman changed the base branch from master to release-6.0.2 February 15, 2024 20:46
@alisman alisman changed the base branch from release-6.0.2 to master February 15, 2024 20:54
@alisman alisman changed the base branch from master to release-6.0.2 February 15, 2024 20:55
@dippindots
Copy link
Member

@alisman I see 3 commits from other prs here, could you cherry-pick your commit 7f01c785b82771a4a6cfc2acf88bf8c42d413108 and force push again?

@alisman
Copy link
Contributor Author

alisman commented Feb 15, 2024

@dippindots fixed. thanks for catching

Copy link
Contributor

@JREastonMarks JREastonMarks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also create one for Master

@inodb inodb added the devops label Feb 16, 2024
@alisman alisman merged commit a337c33 into cBioPortal:release-6.0.2 Feb 16, 2024
11 of 15 checks passed
@alisman alisman deleted the studyTagConf branch February 16, 2024 17:02
JREastonMarks added a commit that referenced this pull request Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants