-
Notifications
You must be signed in to change notification settings - Fork 11.8k
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
Dashboards: Remove emptyDashboardPage feature flag #81188
Dashboards: Remove emptyDashboardPage feature flag #81188
Conversation
@@ -2431,22 +2431,6 @@ exports[`better eslint`] = { | |||
[0, 0, 0, "Do not use any type assertions.", "1"], | |||
[0, 0, 0, "Do not use any type assertions.", "2"] | |||
], | |||
"public/app/features/dashboard/components/AddPanelWidget/AddPanelWidget.test.tsx:5381": [ |
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.
🥳
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.
✅ Feature flag usage removed
✅ Unused code removed
✅ Tests updated
✅ Re-generated docs for FFs without the empty dashboard toggle
Great work @Sergej-Vlasov! Ready to merge!
FYI, the PR is missing a couple of tags:
- Backport tag: If the PR is about a bug fix, that should go into the next patch, we add the proper backport label to the minor where we need to backport these changes. In this case, this is more a cleaning up, so we don't need to backport it.
- Changelog tag: When the change is relevant for a Grafana user, we add the label
add-to-changelog
to add the commit message to the Grafana version changelog automatically. In this case, it is something we could add, so we could add it, it won't hurt and if any customer is using the FF, they can remove it from the configuration
Also, it is missing the CLA acceptation from your side 😄
* remove emptyDashboardPage feature toggle from DashNav * remove emptyDashboardPage feature toggle from NewDashboardWithDS * remove emptyDashboardPage feature toggle from DashboardGrid * remove emptyDashboardPage feature toggle from DashboardModel * remove emptyDashboardPage feature toggle from initDashboard * remove unused AddPanelWidged component * remove add-panel type from test * remove emptyDashboardPage feature flag from registry.go
* remove emptyDashboardPage feature toggle from DashNav * remove emptyDashboardPage feature toggle from NewDashboardWithDS * remove emptyDashboardPage feature toggle from DashboardGrid * remove emptyDashboardPage feature toggle from DashboardModel * remove emptyDashboardPage feature toggle from initDashboard * remove unused AddPanelWidged component * remove add-panel type from test * remove emptyDashboardPage feature flag from registry.go
This PR includes:
emptyDashboardPage
feature flag from the codebaseWhich issue(s) does this PR fix?:
Fixes #81055
Special notes for your reviewer:
Please check that: