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

Support for multiple contributors #2349

Closed
mariapraetzellis opened this issue Jan 13, 2020 · 5 comments
Closed

Support for multiple contributors #2349

mariapraetzellis opened this issue Jan 13, 2020 · 5 comments
Assignees
Labels
maDMP machine-actionable DMPs

Comments

@mariapraetzellis
Copy link

Currently the application allows for 1 PI and 1 "Data Contact Person." In order to support machine actionability of DMPs, we need to expand the number of people that can be linked to a DMP. This could be a repeatable field so that a user could add as many PIs and Data Curators as applicable. This may need a specific own tab in the application. We'd like to make the PI & data contact mandatory.

@mariapraetzellis mariapraetzellis added the maDMP machine-actionable DMPs label Jan 13, 2020
@mariapraetzellis
Copy link
Author

@briri to look at the Credit taxonomy- was a recommendation from a user

@briri
Copy link
Contributor

briri commented Feb 26, 2020

UI work will be done as part of #2408

@sjDCC
Copy link
Contributor

sjDCC commented Apr 24, 2020

I'm happy with the separate tab and think the functionality to add contributors works well but have a couple of suggestions.
Capture
new-contrib

  1. We should rename the tab "Collaborators" not "Contributors" as that's more even. Maria, Magdalena and I discussed this on a Product Manager call

  2. We should pre-enter the plan owner as PI like we did previously on project details and allow them to amend entry. Pre-fill as much as possible from edit profile e.g. ORCID etc

  3. Can we do a look-up integration with the ORCID field instead of this being a text entry?

  4. Are the terms "Data Manager" "Principal Investigator" and "Project Administrator" from CRedIT Taxonomy? May be good to link out to that here or show them as indexed, controlled terms somehow? On the overview table like for RoR entry?

  5. We discussed merging this with the table under "Share" and having the Collaborators tab be where users can list team members and assign plan editing roles. This may be a secondary piece of work to avoid delays now. It could fit under actions feasibly e.g. "Assign privileges" and popup a modal like on admin interface? Or write it into main entry form. To be discussed with UX

credit-link
collab

@briri
Copy link
Contributor

briri commented Apr 24, 2020

Added a new ticket #2481 for these adjustments.

See #2350 for the ORCID question. ORCID does not provide a lookup API. We can pre-populate the value for the user who is logged in. ORCID's workflow though requires that the individual in question asserts that they are associated with the plan. Dryad has solved this by sending out an email to any contributors with a link that lets them assert this information.

For 4) above, those are not the exact terms from CRediT, we can change them to match exactly and can of course easily show the url.

@sjDCC
Copy link
Contributor

sjDCC commented Apr 27, 2020

Request captured in new ticket so closing out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maDMP machine-actionable DMPs
Projects
None yet
Development

No branches or pull requests

3 participants