Skip to content

add function get_user_orcid #142

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

add function get_user_orcid #142

wants to merge 3 commits into from

Conversation

RobLBaker
Copy link
Member

closes #141

@RobLBaker RobLBaker requested a review from wright13 July 25, 2025 21:13
@RobLBaker RobLBaker self-assigned this Jul 25, 2025
@RobLBaker RobLBaker added the enhancement New feature or request label Jul 25, 2025
wright13
wright13 reviewed Jul 25, 2025
View reviewed changes
R/utils.R
#' This is a function to grab a users ORCID from Active Directory. Requires VPN to access AD. If the user does not have an ORCID, returns "NA".
#'
#'
#' @returns Sting. The user's ORCID ID
Copy link
Collaborator

Choose a reason for hiding this comment

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

typo :)

wright13
wright13 approved these changes Jul 25, 2025
View reviewed changes
Copy link
Collaborator

@wright13 wright13 left a comment

Choose a reason for hiding this comment

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

Found a small typo, otherwise looks good! Did you figure out a way to get the orcid for someone who isn't the current user? Or do we just want to wait on updates to the active directory lookup API for that one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wright13 wright13 wright13 approved these changes

Assignees

@RobLBaker RobLBaker

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create a way to access a user's ORCID ID via active directory
2 participants
@RobLBaker @wright13