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

RBAC: Remove legacy AC editor and admin role on new dashboard route #68762

Merged
merged 1 commit into from
May 22, 2023

Conversation

eleijonmarck
Copy link
Contributor

What is this feature?

  • removes the orgroles on new dashboard route for legacy AC

Why do we need this feature?
When RBAC is disabled and you grant a edit permission for viewers in a folder. they are unable to create a new dashboard as it is protected by the orgRoles 'Editor', 'Admin'. This removes this constraints

This allows viewers, even without dashboard permissions, to access the routes. It would enable them to create dashboards but not save them thx to the permissions on the backend.

Who is this feature for?
Azuregrafanasupport

Which issue(s) does this PR fix?:

@eleijonmarck eleijonmarck added add to changelog backport v9.4.x Mark PR for automatic backport to v9.4.x backport v9.5.x Bot will automatically open backport PR backport v10.0.x labels May 22, 2023
@grafana grafana deleted a comment from grafanabot May 22, 2023
@grafana grafana deleted a comment from grafanabot May 22, 2023
@grafana grafana deleted a comment from grafanabot May 22, 2023
@eleijonmarck eleijonmarck marked this pull request as ready for review May 22, 2023 08:38
@eleijonmarck eleijonmarck requested a review from a team as a code owner May 22, 2023 08:38
@eleijonmarck eleijonmarck requested review from joshhunt and eledobleefe and removed request for a team May 22, 2023 08:38
Copy link
Contributor

@gamab gamab left a comment

Choose a reason for hiding this comment

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

LGTM.
You could mention the permission error pop-up they would currently see in the PR description, wdyt?

@eleijonmarck
Copy link
Contributor Author

Yea they are aware of that. It will be fixed from the dashboard squad this week

@eleijonmarck eleijonmarck added this to the 10.0.x milestone May 22, 2023
@eleijonmarck eleijonmarck changed the title RBAC: remove legacy AC editor and admin role on new dashboard route RBAC: Remove legacy AC editor and admin role on new dashboard route May 22, 2023
@eleijonmarck eleijonmarck merged commit 94283f5 into main May 22, 2023
29 checks passed
@eleijonmarck eleijonmarck deleted the eleijonmarck/newdashboard/legacy-orgrole branch May 22, 2023 09:27
grafanabot pushed a commit that referenced this pull request May 22, 2023
…68762)

remove legacy AC editor and admin role on new dashboard route

(cherry picked from commit 94283f5)
grafanabot pushed a commit that referenced this pull request May 22, 2023
…68762)

remove legacy AC editor and admin role on new dashboard route

(cherry picked from commit 94283f5)
grafanabot pushed a commit that referenced this pull request May 22, 2023
…68762)

remove legacy AC editor and admin role on new dashboard route

(cherry picked from commit 94283f5)
eleijonmarck added a commit that referenced this pull request May 22, 2023
…d route (#68776)

RBAC: Remove legacy AC editor and admin role on new dashboard route (#68762)

remove legacy AC editor and admin role on new dashboard route

(cherry picked from commit 94283f5)

Co-authored-by: Eric Leijonmarck <eric.leijonmarck@gmail.com>
eleijonmarck added a commit that referenced this pull request May 22, 2023
…rd route (#68777)

RBAC: Remove legacy AC editor and admin role on new dashboard route (#68762)

remove legacy AC editor and admin role on new dashboard route

(cherry picked from commit 94283f5)

Co-authored-by: Eric Leijonmarck <eric.leijonmarck@gmail.com>
eleijonmarck added a commit that referenced this pull request May 22, 2023
…d route (#68775)

RBAC: Remove legacy AC editor and admin role on new dashboard route (#68762)

remove legacy AC editor and admin role on new dashboard route

(cherry picked from commit 94283f5)

Co-authored-by: Eric Leijonmarck <eric.leijonmarck@gmail.com>
ryantxu pushed a commit that referenced this pull request May 22, 2023
…68762)

remove legacy AC editor and admin role on new dashboard route
@zerok zerok modified the milestones: 10.0.x, 10.0.0-preview, 10.1.x May 31, 2023
kminehart pushed a commit that referenced this pull request Jun 22, 2023
…d route (#68776)

RBAC: Remove legacy AC editor and admin role on new dashboard route (#68762)

remove legacy AC editor and admin role on new dashboard route

(cherry picked from commit 94283f5)

Co-authored-by: Eric Leijonmarck <eric.leijonmarck@gmail.com>
@ricky-undeadcoders ricky-undeadcoders modified the milestones: 10.1.x, 10.1.0 Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add to changelog area/frontend backport v9.4.x Mark PR for automatic backport to v9.4.x backport v9.5.x Bot will automatically open backport PR backport v10.0.x type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants