Conversation
|
Blocked; should not be merged until CC4K v0.5 is ready for production. |
|
/preview |
1 similar comment
|
/preview |
|
/destroy |
|
/preview |
There was a problem hiding this comment.
Pull request overview
Updates St. Jude Cloud documentation and downloadable artifacts for the CC4K v0.5 release, and adds a new metadata field description for “About our data”.
Changes:
- Add CC4K v0.5 documentation section and publish the v0.5 Excel download.
- Add
ped_cancer_diagnosis_categoryto the metadata field table and reviseattr_diagnosis_groupwording. - Add a Flux HelmRelease manifest for the PR131 preview environment.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| public/files/cc4k/Official_CC4K_v0.5.xlsx | Adds the CC4K v0.5 downloadable spreadsheet artifact. |
| deployment/preview/pr131/app.yaml | Adds PR-specific preview HelmRelease configuration (docs-pr131). |
| content/3.genomics-platform/2.about-our-data/3.metadata-and-clinical-information.md | Documents new diagnosis-category metadata and revises diagnosis-group description. |
| content/2.cc4k/1.index.md | Adds CC4K v0.5 release notes and download link. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
content/3.genomics-platform/2.about-our-data/3.metadata-and-clinical-information.md
Outdated
Show resolved
Hide resolved
…8a-231 to pr131 [skip ci]
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…inical-information.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
mkirk111
force-pushed
the
cc4kv0.5-1.22.25
branch
from
b7459d5 to
3a1922a
Compare
|
/preview |
|
/destroy |
content/2.cc4k/1.index.md
Outdated
|
|
||
| Submitted: March 11, 2026 | ||
|
|
||
| Click [here](/files/cc4k/Official_CC4K_v0.5.xlsx) {target="_blank"} to download. |
There was a problem hiding this comment.
There's a space before {target="_blank"} here that the other versions don't have — I think this will cause it to render as literal text rather than being applied to the link.
| Click [here](/files/cc4k/Official_CC4K_v0.5.xlsx) {target="_blank"} to download. | |
| Click [here](/files/cc4k/Official_CC4K_v0.5.xlsx){target="_blank"} to download. |
Co-authored-by: Clay McLeod <3411613+claymcleod@users.noreply.github.com>
claymcleod
left a comment
claymcleod
left a comment
There was a problem hiding this comment.
All feedback addressed — looks good!
Update university docs with the following: