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
Feat: Feature toggle admin page frontend interface #72164
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.
LGTM as a first pass! Please make the change requested by Josh before merging.
Let's do a follow-up task to use the other table impl.
dismissing as there are other requested changes. Maybe we should use the fancy table component here instead?
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.
👍🏻 Great stuff with the RTK Query usage!
Nice to see InteractiveTable get more usage! Eager for any feedback you might have from trying it.
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.
LGTM - got a walk-through from @IbrahimCSAE showing me all the comments and resulting changes.
We will be iterating on this page, so if you have any further comments, please share in #proj-feature-toggle-admin-page in slack
* feature toggles admin page proto * feature toggle admin page proto * keep phase 1 code only * latest update with api * fix * fix * add correct premissions in admin.go * move behind toggle * Use InteractiveTable * guard behind feature toggle * use RTK * route in api.go * fixes
* feature toggles admin page proto * feature toggle admin page proto * keep phase 1 code only * latest update with api * fix * fix * add correct premissions in admin.go * move behind toggle * Use InteractiveTable * guard behind feature toggle * use RTK * route in api.go * fixes
What is this feature?
Added a new admin page to allow admin users to view a list of feature toggles and their status on their grafana instance.
Why do we need this feature?
We are creating a Feature Toggle Management page within grafana, where the user can see (and update in the future) the current feature toggles state.
Who is this feature for?
Grafana Administrators
Which issue(s) does this PR fix?:
Fixes #70784 & https://github.com/grafana/grafana-operator-experience-squad/issues/608