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

Allow users to specify their ORCID and/or ISNI ID in their user account #923

Closed
pdurbin opened this issue Sep 15, 2014 · 21 comments
Closed

Comments

@pdurbin
Copy link
Member

pdurbin commented Sep 15, 2014

From a meeting with @mcrosas @scolapasta and @pdurbin we should allow users to specify their ORCID and/or ISNI ID for their user account at sign up or later when editing their account.

This is for user accounts, not datasets, per https://docs.google.com/a/harvard.edu/document/d/1bZjEsejCaw1U6zNkER7iaZbysmXRKmonp9tKLUn-4LI/edit?usp=sharing

See also #829.

@pdurbin pdurbin added this to the In Review - Dataverse 4.0 milestone Sep 15, 2014
@pdurbin pdurbin changed the title Allow users to specify their ORCID and/or ISNE ID in their user account Allow users to specify their ORCID and/or ISNI ID in their user account Sep 15, 2014
@eaquigley eaquigley modified the milestones: Beta 10 - Dataverse 4.0, In Review - Dataverse 4.0 Sep 26, 2014
@eaquigley eaquigley modified the milestones: Dataverse 4.0: Final, Beta 10 - Dataverse 4.0 Dec 8, 2014
@scolapasta
Copy link
Contributor

Is this for built in users or also shibboleth? currently, I don't believe we allow users to add any extra info (yes, including usernames) for shibboleth )or other external).

@scolapasta scolapasta assigned eaquigley and unassigned scolapasta Jan 15, 2015
@pdurbin
Copy link
Member Author

pdurbin commented Jan 16, 2015

Is this for built in users or also shibboleth?

All users should be treated equally.

@scolapasta scolapasta modified the milestones: Dataverse 4.0: Final, TEMP, In Review - Dataverse 4.0 Jan 23, 2015
@eaquigley
Copy link
Contributor

Since 4.0 is now coming out towards the end of March and I am already rethinking the Logging In process, I would like to add this field in to the registration form.

@eaquigley eaquigley modified the milestones: Beta 13 - Dataverse 4.0, In Review - Dataverse 4.0 Feb 9, 2015
@posixeleni
Copy link
Contributor

@eaquigley would this field also be pre-populated in the Dataset form like we do for the author name? This would save the user a lot of time having to re-enter the same information twice (or multiple times if they have many datasets).

@eaquigley
Copy link
Contributor

@posixeleni yes ma'am, anything from the user sign up that relates to dataset metadata needs to be pre-populated.

@pdurbin
Copy link
Member Author

pdurbin commented Feb 9, 2015

If we're going to revisit the registration form, can we please make it so all users can have their "Affiliation" and "Position" stored in the database? (Or simply remove affiliation and position altogether?) If we need a separate ticket to revisit the builtin user table (where affiliation and position are stored) vs. the authenticateduser table, that's fine. To fix #945 and #457 and #965 the authenticated user table should probably have separate firstName and lastName columns.

Obviously, all users should be able to specify their ORCID or other ID, not just builtin users.

@suenjedt
Copy link

An update from the meeting:

  • There is a preference from the ORCID side to start with Oauth and Phase 2
  • However, there are examples, e.g. by ANDS where Phase I equivalent widgets are built http://developers.ands.org.au/widgets/orcid_widget/.
  • Discussion shows preference to start with Phase 1 as planned, with suggestions for ORCID records (similar to ANDS integration)

quick update: I asked our colleagues at ANDS, who could not give me any additional information in regards to their ORCID widget that we looked into during our ORCID meeting. The other tool that was mentioned has not been implemented yet. More details (e.g. code) to follow from my CERN colleagues.


My summary from the meeting and follow up conversations: there are different models for implementations (e.g. publisher's manuscript submission systems, repositories or community portals such as ADS). ORCID prefers an Oauth integration for most, or all. However, it seems feasible to start with our planned Phase 1 and I, personally, dont see why we should not move as planned. Phase 1 is already an enhancement to the current state and Phase 2 requires more discussion and thus would happen much further down the road.

You are of course welcome to revise my summary.

[However, what is probably needed, is a conversation about Oauth as well, non?]

@pdurbin
Copy link
Member Author

pdurbin commented Jul 13, 2015

I've listed OAuth as an interest of the Dataverse Auth Community Group: https://docs.google.com/document/d/1xbWkj_OIk13gKe7jleNXVvakZErvHLA8JRBvLz8Fe8Q/edit?usp=sharing

@mercecrosas mercecrosas modified the milestones: In Design, In Review Nov 30, 2015
@pdurbin
Copy link
Member Author

pdurbin commented Jan 6, 2016

what are the technical details for allowing shibboleth users to add in their ORCID?

From an email I sent last summer:

Eventually, we may be able to get ORCID IDs from Shibboleth in the form of a SAML assertion of an "eduPersonOrcid" attribute per the (unfinalized) spec linked from https://spaces.internet2.edu/display/macedir/ORCID+and+other+researcher+IDs

This was mentioned yesterday in the IT Summit talk "Enabling Cross-University Collaboration with Harvard IAM: TIER, InCommon, and Grouper"

@pdurbin
Copy link
Member Author

pdurbin commented Oct 3, 2016

This issue is highly related to #3338 which is currently under development.

@djbrooke
Copy link
Contributor

I'm going to close this in favor of #3338.

@djbrooke
Copy link
Contributor

djbrooke commented Oct 31, 2016

Talked with @scolapasta, got some more info, reopening. Oops.

#3338 deals with logging in with / converting an account to use ORCID. This issue deals with maintaining an ORCID association for non-ORCID accounts AND using an ORCID throughout the application.

If this is worked on as part of #3338, we'll revisit closing/combining.

@djbrooke djbrooke reopened this Oct 31, 2016
@djbrooke
Copy link
Contributor

djbrooke commented Nov 2, 2016

This is out of scope for #3338 and will live on a separate issue.

@pdurbin
Copy link
Member Author

pdurbin commented Jun 25, 2017

Out of scope for #3507 too.

@pdurbin
Copy link
Member Author

pdurbin commented Jun 28, 2017

Today I stubbed out a getOrcidId method on AuthenticatedUser in ca33857 but it's only for users who log in via ORCID.

@pdurbin
Copy link
Member Author

pdurbin commented Jun 28, 2017

I'm closing this until we decide to start working on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants