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

Add specific permission to administer data-dictionary settings #3865

Conversation

tiffneybare
Copy link
Collaborator

@tiffneybare tiffneybare commented Nov 3, 2022

Some sites may not want users who manage data publishing to have the very broad 'Administer site configuration' permission in order to access the data dictionary settings screen.

QA Steps

  • Add the new administer data dictionary settings permission to an existing role
  • Create a user with that role
  • Log in as the new user
  • Confirm you have access to the following page: /admin/dkan/data-dictionary/settings

@janette
Copy link
Member

janette commented Nov 3, 2022

@tiffneybare I think it would be better to modify the role permissions on the project as we don't want this exposed to roles that just need access to edit content.

@tiffneybare tiffneybare closed this Nov 8, 2022
@tiffneybare tiffneybare reopened this Nov 8, 2022
@janette janette changed the title Updating Data Publisher permissions to include data-dictionary settings Add specific permission to administer data-dictionary settings Nov 14, 2022
Copy link
Member

@janette janette left a comment

Choose a reason for hiding this comment

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

Adding "Metastore" to the permission titles to follow the pattern set by the Datastore and Harvest modules.

@janette janette merged commit 6651685 into 2.x Nov 14, 2022
@janette janette deleted the data-publisher-permissions-to-include-data-dictionary-settings-page branch May 19, 2023 14:25
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.

None yet

2 participants