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
docs(xmlupload): better explanation of permissions (DEV-969) #196
docs(xmlupload): better explanation of permissions (DEV-969) #196
Conversation
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.
Cool, thanks.
docs/dsp-tools-xmlupload.md
Outdated
|
||
The following access rights are defined by the DSP platform which apply to either a resource or a field: | ||
The following access rights are defined by the DSP platform: |
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.
"DSP platform" is actually "doppelt gemoppelt" because DSP already means DaSCH Service Platform... If you want to adjust that here (I suggest just using "DSP"), could you go through all of the documentation to see if there are more occurrences of that case?
- `M` _modifiy permission_: The user may modify the element, but may not delete it. | ||
- `D` _delete permission_: The user is allowed to delete the element. | ||
- `CR` _change right permission_: The user may change the permission of the element. | ||
- (no right): If no permission is defined for a certain group of users, these users cannot view any resources/values. |
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.
Adding this makes it much clearer now!
Co-authored-by: irinaschubert <irina.schubert@dasch.swiss>
Kudos, SonarCloud Quality Gate passed! |
docs/dsp-tools-xmlupload.md
Outdated
@@ -437,6 +465,7 @@ Attributes: | |||
- `permissions`: ID or a permission set (optional, but if omitted, very restricted default permissions apply) |
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.
@irinaschubert Here I am not certain what "ID or a permission set" means. I know only one possibility of adding a permission, but here it sounds if it were 2 possibilities...
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.
Good question, I don't know what this means. I would just use "ID" or "permission ID" or something.
…f-permissions' into wip/dev-969-better-explanation-of-permissions
resolves DEV-969