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.

Sign up for GitHub

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

Jump to bottom

workspace toolbar: the n-window selector should be disabled according to permissions #1845

Closed
oliver-sanders opened this issue Jun 27, 2024 · 0 comments · Fixed by #1979
Closed

workspace toolbar: the n-window selector should be disabled according to permissions #1845

oliver-sanders opened this issue Jun 27, 2024 · 0 comments · Fixed by #1979
Assignees
@oliver-sanders
Labels
bug Something isn't working small
Milestone
2.7.0

Comments

@oliver-sanders
Copy link
Member

Small low-priority issue with the n-window selector.

If the user does not have the permission to modify the n-window (control level), then it remains editable making it look like the user can change it, which they cant.
@oliver-sanders oliver-sanders added bug Something isn't working small labels Jun 27, 2024
@oliver-sanders oliver-sanders added this to the 2.6.0 milestone Jun 27, 2024
@oliver-sanders oliver-sanders modified the milestones: 2.6.0, 2.7.0 Oct 9, 2024
oliver-sanders added a commit to oliver-sanders/cylc-ui that referenced this issue Oct 29, 2024
@oliver-sanders
toolbar: disable n-window selector if appropriate
d71ee83 
* Closes cylc#1845
* Disable the n-window selector if the authenticated user does not
  have the required permissions to run the setGraphWindowExtent
  mutation.
@oliver-sanders oliver-sanders mentioned this issue Oct 29, 2024
Merged
8 tasks
@oliver-sanders oliver-sanders self-assigned this Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oliver-sanders oliver-sanders

Labels
bug Something isn't working small
Projects
None yet
Milestone
2.7.0
Development

Successfully merging a pull request may close this issue.

toolbar: disable n-window selector if appropriate oliver-sanders/cylc-ui
1 participant
@oliver-sanders