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

ShortCut-3897: add external user #1027

Merged
merged 1 commit into from
Feb 11, 2025
Merged

ShortCut-3897: add external user #1027

merged 1 commit into from
Feb 11, 2025

Conversation

swalker2m
Copy link
Contributor

Adds a new ProgramUserRole, the External user. External users would be associated with a program, have the same attributes as other program users (name, email, ORCiD, etc.), be invited and accept invitations, etc. like a user in any other role. The difference is that external users are not granted permission to see or edit the corresponding program.

The reason to add an External role is for a new program user attribute, hasDataAccess. When true, the user will be allowed to download the program's data. ShortCut 3897 prescribes a data sharing list and applies it even to COIs. This means that a user may be a COI with full edit permission and yet not allowed to download the data. Conversely, a user may be external and not allowed to see the program and yet have permission to download the data.

A database query will be provided for the GOA. Given an ORCiD, GOA will check with the database to see whether the user is allowed to download data.

@mergify mergify bot added the core label Feb 11, 2025
@tpolecat
Copy link
Member

This seems reasonable to me. 👍

@swalker2m swalker2m merged commit 6278c56 into main Feb 11, 2025
9 checks passed
@swalker2m swalker2m deleted the sc-3897-download-access branch February 11, 2025 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants