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

adjust UI to consider and disable/grey-out non applicable permissions on resources by type #503

Open
fmigneault opened this issue Feb 3, 2022 · 0 comments
Assignees
Labels
enhancement Improvements in term of performance or behaviour ui Something related to the UI operations or display

Comments

@fmigneault
Copy link
Collaborator

fmigneault commented Feb 3, 2022

relates to #499

For example:
image

The GetCapabilities does not apply to anything else than the service itself. Trying to set it on any underlying resource will yield an error on save. Using the service definition, it should be possible to pre-filter the permissions to gray-out the col/row combinations that cannot be applied.

Similarly, the Execute can only be applied for a Process resource (not Workspace or Layer).
However, Execute can still be applied on the service instead of the resource to allow it recursively on all nested Process resources.

The UI implementation should not try to resolve itself the valid combinations. The API should return the applicable permissions, and the UI would simply gray-out/disable the combobox selectors when not applicable. The API should already return this information (or at least it should be closely available) since the permission creation is already checked for valid cases and raises the error otherwise.

@fmigneault fmigneault added enhancement Improvements in term of performance or behaviour ui Something related to the UI operations or display labels Feb 3, 2022
@fmigneault fmigneault self-assigned this Feb 3, 2022
@fmigneault fmigneault changed the title adjust UI to consider above and disable/grey-out non applicable permissions on resources by type adjust UI to consider and disable/grey-out non applicable permissions on resources by type Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvements in term of performance or behaviour ui Something related to the UI operations or display
Projects
None yet
Development

No branches or pull requests

1 participant