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

Update news 4.21 #1153

Merged
merged 12 commits into from
Sep 25, 2024
Merged

Update news 4.21 #1153

merged 12 commits into from
Sep 25, 2024

Conversation

jfkonecn
Copy link
Contributor

@jfkonecn jfkonecn commented Sep 13, 2024

Related to https://github.com/oncokb/oncokb-pipeline/issues/531

  • Update data release date
  • Update index.html and oncokb_summary.png

Comment on lines 304 to 308
<PMIDLink pmids="38410486" />
<PMIDLink pmids="37012551" />
<PMIDLink pmids="26554404" />
<PMIDLink pmids="38032104" />
<PMIDLink pmids="27483357" />
Copy link
Collaborator

@calvinlu3 calvinlu3 Sep 13, 2024

Choose a reason for hiding this comment

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

You can directly pass a comma separated string to pmids

[
'3A',
'ALK',
'F1174L/C, F1245Y/V, R1275Q/L',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you verify if this will be converted to alteration page links? If not, we might need to use WithSeparator.

Copy link
Collaborator

@calvinlu3 calvinlu3 left a comment

Choose a reason for hiding this comment

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

@jfkonecn I only reviewed the last 3 commits of this PR

Copy link
Collaborator

Choose a reason for hiding this comment

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

This file may need updating once we do a refresh tomorrow since we added back an alteration that was deleted by accident.

@@ -15,15 +14,15 @@
<meta name="twitter:site" content="@oncokb">
<meta name="twitter:title" content="OncoKB™ - MSK's Precision Oncology Knowledge Base">
<meta name="twitter:description" content="OncoKB™ is a precision oncology knowledge base developed at Memorial Sloan Kettering Cancer Center that contains biological and clinical information about genomic alterations in cancer.">
<meta name="twitter:image" content="https://www.oncokb.org/content/images/oncokb_summary.png?20240704">
<meta name="twitter:image" content="https://www.oncokb.org/content/images/oncokb_summary.png?20240920">
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's update date before merging

@jfkonecn jfkonecn marked this pull request as ready for review September 24, 2024 19:19
<meta name="twitter:url" content="https://www.oncokb.org/">

<meta property="og:site_name" content="OncoKB™">
<meta property="og:type" content="website">
<meta property="og:url" content="https://www.oncokb.org/">
<meta property="og:title" content="OncoKB™ - MSK's Precision Oncology Knowledge Base">
<meta property="og:description" content="OncoKB™ is a precision oncology knowledge base developed at Memorial Sloan Kettering Cancer Center that contains biological and clinical information about genomic alterations in cancer.">
<meta property="og:image" content="https://www.oncokb.org/content/images/oncokb_summary.png?20240704">
<meta property="og:image" content="https://www.oncokb.org/content/images/oncokb_summary.png?20240920">
Copy link
Collaborator

Choose a reason for hiding this comment

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

Update date here too

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure if it was initially intended to include the text Power by the clinical expertise...., but lets include that in the screenshot.

Copy link
Collaborator

@calvinlu3 calvinlu3 left a comment

Choose a reason for hiding this comment

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

Looks good. Just some merge conflicts that need to be resolve it seems.

@jfkonecn jfkonecn merged commit f96b287 into oncokb:master Sep 25, 2024
6 checks passed
@jfkonecn jfkonecn deleted the feature/v4.21 branch September 25, 2024 10:28
jfkonecn added a commit to jfkonecn/oncokb-public that referenced this pull request Oct 7, 2024
* Update news 4.20 (oncokb#1142)

---------

Co-authored-by: Calvin Lu <[email protected]>
Co-authored-by: Hongxin <[email protected]>
Co-authored-by: oncokb-bot <[email protected]>
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.

3 participants