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

WC-161: update citation doi #1031

Draft
wants to merge 1 commit into
base: task/digital-rocks
Choose a base branch
from

Conversation

van-go
Copy link
Contributor

@van-go van-go commented Dec 4, 2024

Overview

Masa wants the only link to be the DOI, https://www.doi.org/##########
Do not show digitalrocks link
Citations in Published projects:
Currently it references a link to the DR portal, but should reference/cite the DOI link instead.
WMA will add the DOI to the published info.
If the data path changes, we update the DOI info.
Masa prefers we expose the DOI link only, not the path to the portal (in case it changes).

Related

Changes

Updated ReviewAuthors.jsx to use a doi if available:
// Use DOI if available, fallback to project URL
const projectUrl = project.doi
? https://www.doi.org/${project.doi}
: DOI link will appear after publication;

Testing

  1. For an unpublished project, while in the publication pipeline, continue to "Review Authors" modal. Make sure 'DOI link will appear after publication' appears.
  2. For published projects, under "Cite This Data:" make sure the doi.org/##### link shows and works.
  3. For published projects, click "View Additional Citations" and make sure the correct doi.org link appears and works.

UI

Screen Shot 2024-12-04 at 11 08 52 AM

Notes

@van-go van-go changed the base branch from main to task/digital-rocks December 4, 2024 17:13
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.

1 participant