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

Task/kbdev 846 fix pubmed URLs to new pubmed URL #128

Merged
merged 2 commits into from
Nov 3, 2023

Conversation

sshugsc
Copy link
Contributor

@sshugsc sshugsc commented Nov 3, 2023

update pubmed url to https://pubmed.ncbi.nlm.nih.gov

@sshugsc sshugsc changed the title update pubmed url task/KBDEV 846 fix pubmed URLs to new pubmed URL Nov 3, 2023
@sshugsc sshugsc changed the title task/KBDEV 846 fix pubmed URLs to new pubmed URL Task/kbdev 846 fix pubmed URLs to new pubmed URL Nov 3, 2023
@sshugsc sshugsc self-assigned this Nov 3, 2023
src/sources.js Outdated
@@ -242,7 +242,7 @@ const sources = {
description: 'pubmed comprises more than 29 million citations for biomedical literature from medline, life science journals, and online books. citations may include links to full-text content from pubmed central and publisher web sites',
Copy link
Member

Choose a reason for hiding this comment

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

Since the number of citations keep increasing, I suggest we also change the description.
From the about page: "PubMed is a free resource supporting the search and retrieval of biomedical and life sciences literature with the aim of improving health–both globally and personally."

This comment has been minimized.

@mathieulemieux mathieulemieux self-requested a review November 3, 2023 22:26
@sshugsc sshugsc merged commit 77feb33 into develop Nov 3, 2023
5 checks passed
Copy link

github-actions bot commented Nov 3, 2023

Unit Test Results

0 files  ±0  0 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 77feb33. ± Comparison against base commit e7fa24e.

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.

2 participants