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

Feat/global cohorts feature sync - based on PR by Rich #44

Merged
merged 5 commits into from
May 9, 2024

Conversation

pieterlukasse
Copy link

@pieterlukasse pieterlukasse commented May 6, 2024

Pulling in changes from OHDSI#2929

Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/VADC-1063

New Features

Deployment changes

  • A new configuration item needs to be added to js/config-local.js (according to doc from upstream PR):
    • configLocal.permissionManagementRoleId: By default set to an empty string ('') indicating that all users can share globally. Optionally, it can be set to the id of the a non-system role used to restrict the ability to share artifacts globally to a specific set of users

…ranting write access to a global shared artifact reader role that would be given to all users. Next steps are to 1) make this configurable, 2) allow users to configure the global author role if they want only some users to be able to grant global read access, and 3) set up global read role for all users as a default assigned system role
…'public' role since this would remove the need to change WebAPI to add a new system role that pretty much duplicates 'public'
@pieterlukasse pieterlukasse changed the title Feat/global cohorts feature sync - base on PR by Rich Feat/global cohorts feature sync - based on PR by Rich May 6, 2024
@pieterlukasse pieterlukasse requested a review from m0nhawk May 6, 2024 15:41
Copy link

@m0nhawk m0nhawk left a comment

Choose a reason for hiding this comment

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

👍

@pieterlukasse pieterlukasse merged commit c2f249c into 2.15.0-DEV May 9, 2024
3 checks passed
@pieterlukasse pieterlukasse deleted the feat/global_cohorts_feature_sync branch May 9, 2024 13:59
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.

3 participants