-
Notifications
You must be signed in to change notification settings - Fork 18
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: Allow external iris for property and resource class when updating doap (DEV-4341) #3425
base: main
Are you sure you want to change the base?
Conversation
webapi/src/main/scala/org/knora/webapi/responders/admin/PermissionsResponder.scala
Outdated
Show resolved
Hide resolved
…:permissionIri/for-what allowing to update the restrictions at a single endpoint
I have added a new endpoint Please have another review. |
…nPart check that token and name are consistent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here I already leave some comments. More is to follow.
Co-authored-by: Johannes Nussbaum <[email protected]>
Co-authored-by: Johannes Nussbaum <[email protected]>
Co-authored-by: Johannes Nussbaum <[email protected]>
Co-authored-by: Johannes Nussbaum <[email protected]>
Co-authored-by: Johannes Nussbaum <[email protected]>
Co-authored-by: Johannes Nussbaum <[email protected]>
deprecated: Deprecate endpoints for updating a Default Access Permission in favor of newly introduces
PUT /admin/permissions/doap/:permissionIri
endpointDeprecated endpoints:
PUT /admin/permissions/:permissionIri/resourceClass
PUT /admin/permissions/:permissionIri/property
PUT /admin/permissions/:permissionIri/group
(deprecated only for DOAPs)PUT /admin/permissions/:permissionIri/hasPermission
(deprecated only for DOAPs)Pull Request Checklist
Task Description/Number
Issue Number: DEV-
PR Type
Basic requirements for bug fixes and features
Does this PR introduce a breaking change?