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

Requested Changes [post-workshop] for the "Add Dataset" View #1064

Open
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 Feb 25, 2025

Overview

Related

Changes

Add the following placeholder text to the identified input field:

  • Dataset Title: “The title should be descriptive and distinctive from related publications.”

  • Description: “Provide 200-300 words clearly describing the data as an independent output; do not copy related publication abstract.”

  • The label “Members” should be changed to “Authors”.

PI question about the Add Dataset view: We are not clear what is the meaning of the “Role”, especially since here we cannot add anyone else.

Testing

  1. click "+Add" --> choose "Dataset"
  2. Check modal for expected changes

UI

Screen Shot 2025-02-25 at 11 45 35 AM

Notes

changing "members" to "authors" will reflect everywhere DataFilesProjectMembers.jsx is used.

@van-go van-go requested a review from shayanaijaz February 25, 2025 17:50
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