-
Notifications
You must be signed in to change notification settings - Fork 12.3k
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
Service Accounts: Enable adding folder, dashboard and data source permissions to service accounts #76133
Conversation
Co-authored-by: ievaVasiljeva <ieva.vasiljeva@grafana.com>
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.
Whoop whoop 🚢
Two small comments (one code comment and one below), but apart from that all looks good!
Nit: maybe spell out the title? So ServiceAccounts: Add SAs to managed permissions
-> Service Accounts: Enable adding folder, dashboard and data source permissions to service accounts
. Just cause it goes in the change log, and people won't know what "managed permissions" means.
Co-authored-by: Ieva <ieva.vasiljeva@grafana.com>
Great to see the change - thank you! Are you planning to also have a What's new section for this one? |
We can. it's not really new as we never stopped it in the backend and we added terraform support for it. |
What is this feature?
Allow adding Service accounts to Datasource, Dashboard and Folder Permissions.
Why do we need this feature?
[Add a description of the problem the feature is trying to solve.]
Who is this feature for?
[Add information on what kind of user the feature is for.]
Which issue(s) does this PR fix?:
Fixes #
Special notes for your reviewer:
Please check that: