From 4fbd9a55b27c20b86306c7e4be5ca03e79543918 Mon Sep 17 00:00:00 2001 From: Taewoo K Date: Mon, 6 Nov 2023 15:21:15 -0500 Subject: [PATCH] Remove feature toggle data source page header (#77751) * remove feature toggle:dataSourcePageHeader * remove feature flag: dataSourcePageHeader * remove unused prop:alertingSupported --- .../feature-toggles/index.md | 1 - .../src/types/featureToggles.gen.ts | 1 - pkg/services/featuremgmt/registry.go | 7 ------- pkg/services/featuremgmt/toggles_gen.csv | 1 - pkg/services/featuremgmt/toggles_gen.go | 4 ---- .../connections/pages/EditDataSourcePage.tsx | 20 +------------------ .../components/BasicSettings.test.tsx | 1 - .../datasources/components/BasicSettings.tsx | 13 +----------- .../datasources/components/EditDataSource.tsx | 5 ----- .../features/datasources/state/navModel.ts | 5 ----- 10 files changed, 2 insertions(+), 56 deletions(-) diff --git a/docs/sources/setup-grafana/configure-grafana/feature-toggles/index.md b/docs/sources/setup-grafana/configure-grafana/feature-toggles/index.md index a07d989884a..80cd6398105 100644 --- a/docs/sources/setup-grafana/configure-grafana/feature-toggles/index.md +++ b/docs/sources/setup-grafana/configure-grafana/feature-toggles/index.md @@ -74,7 +74,6 @@ Some features are enabled by default. You can disable these feature by setting t | `refactorVariablesTimeRange` | Refactor time range variables flow to reduce number of API calls made when query variables are chained | | `faroDatasourceSelector` | Enable the data source selector within the Frontend Apps section of the Frontend Observability | | `enableDatagridEditing` | Enables the edit functionality in the datagrid panel | -| `dataSourcePageHeader` | Apply new pageHeader UI in data source edit page | | `sqlDatasourceDatabaseSelection` | Enables previous SQL data source dataset dropdown behavior | | `awsAsyncQueryCaching` | Enable caching for async queries for Redshift and Athena. Requires that the `useCachingService` feature toggle is enabled and the datasource has caching and async query support enabled | | `splitScopes` | Support faster dashboard and folder search by splitting permission scopes into parts | diff --git a/packages/grafana-data/src/types/featureToggles.gen.ts b/packages/grafana-data/src/types/featureToggles.gen.ts index 73be2840664..e38ee6933b3 100644 --- a/packages/grafana-data/src/types/featureToggles.gen.ts +++ b/packages/grafana-data/src/types/featureToggles.gen.ts @@ -86,7 +86,6 @@ export interface FeatureToggles { advancedDataSourcePicker?: boolean; faroDatasourceSelector?: boolean; enableDatagridEditing?: boolean; - dataSourcePageHeader?: boolean; extraThemes?: boolean; lokiPredefinedOperations?: boolean; pluginsFrontendSandbox?: boolean; diff --git a/pkg/services/featuremgmt/registry.go b/pkg/services/featuremgmt/registry.go index 06726c579e4..cf9c374a9bc 100644 --- a/pkg/services/featuremgmt/registry.go +++ b/pkg/services/featuremgmt/registry.go @@ -497,13 +497,6 @@ var ( Stage: FeatureStagePublicPreview, Owner: grafanaBiSquad, }, - { - Name: "dataSourcePageHeader", - Description: "Apply new pageHeader UI in data source edit page", - FrontendOnly: true, - Stage: FeatureStagePublicPreview, - Owner: enterpriseDatasourcesSquad, - }, { Name: "extraThemes", Description: "Enables extra themes", diff --git a/pkg/services/featuremgmt/toggles_gen.csv b/pkg/services/featuremgmt/toggles_gen.csv index d7936460069..aee638498e0 100644 --- a/pkg/services/featuremgmt/toggles_gen.csv +++ b/pkg/services/featuremgmt/toggles_gen.csv @@ -67,7 +67,6 @@ enableElasticsearchBackendQuerying,GA,@grafana/observability-logs,false,false,fa advancedDataSourcePicker,GA,@grafana/dashboards-squad,false,false,false,true faroDatasourceSelector,preview,@grafana/app-o11y,false,false,false,true enableDatagridEditing,preview,@grafana/grafana-bi-squad,false,false,false,true -dataSourcePageHeader,preview,@grafana/enterprise-datasources,false,false,false,true extraThemes,experimental,@grafana/grafana-frontend-platform,false,false,false,true lokiPredefinedOperations,experimental,@grafana/observability-logs,false,false,false,true pluginsFrontendSandbox,experimental,@grafana/plugins-platform-backend,false,false,false,true diff --git a/pkg/services/featuremgmt/toggles_gen.go b/pkg/services/featuremgmt/toggles_gen.go index 2a8030b0bf5..70f2a403d79 100644 --- a/pkg/services/featuremgmt/toggles_gen.go +++ b/pkg/services/featuremgmt/toggles_gen.go @@ -279,10 +279,6 @@ const ( // Enables the edit functionality in the datagrid panel FlagEnableDatagridEditing = "enableDatagridEditing" - // FlagDataSourcePageHeader - // Apply new pageHeader UI in data source edit page - FlagDataSourcePageHeader = "dataSourcePageHeader" - // FlagExtraThemes // Enables extra themes FlagExtraThemes = "extraThemes" diff --git a/public/app/features/connections/pages/EditDataSourcePage.tsx b/public/app/features/connections/pages/EditDataSourcePage.tsx index 28221e47e8a..bc4053609d7 100644 --- a/public/app/features/connections/pages/EditDataSourcePage.tsx +++ b/public/app/features/connections/pages/EditDataSourcePage.tsx @@ -1,31 +1,13 @@ import * as React from 'react'; import { useLocation, useParams } from 'react-router-dom'; -import { config } from '@grafana/runtime'; -import { Page } from 'app/core/components/Page/Page'; import DataSourceTabPage from 'app/features/datasources/components/DataSourceTabPage'; -import { EditDataSource } from 'app/features/datasources/components/EditDataSource'; -import { EditDataSourceActions } from 'app/features/datasources/components/EditDataSourceActions'; - -import { useDataSourceSettingsNav } from '../hooks/useDataSourceSettingsNav'; export function EditDataSourcePage() { const { uid } = useParams<{ uid: string }>(); const location = useLocation(); const params = new URLSearchParams(location.search); const pageId = params.get('page'); - const dataSourcePageHeader = config.featureToggles.dataSourcePageHeader; - const { navId, pageNav } = useDataSourceSettingsNav(); - if (dataSourcePageHeader) { - return ; - } - - return ( - }> - - - - - ); + return ; } diff --git a/public/app/features/datasources/components/BasicSettings.test.tsx b/public/app/features/datasources/components/BasicSettings.test.tsx index 7a869dc8beb..18c654deb35 100644 --- a/public/app/features/datasources/components/BasicSettings.test.tsx +++ b/public/app/features/datasources/components/BasicSettings.test.tsx @@ -11,7 +11,6 @@ const setup = () => { isDefault: false, onDefaultChange: jest.fn(), onNameChange: jest.fn(), - alertingSupported: false, }; return render(); diff --git a/public/app/features/datasources/components/BasicSettings.tsx b/public/app/features/datasources/components/BasicSettings.tsx index 52228cc2228..0e630a7031b 100644 --- a/public/app/features/datasources/components/BasicSettings.tsx +++ b/public/app/features/datasources/components/BasicSettings.tsx @@ -3,7 +3,6 @@ import React from 'react'; import { GrafanaTheme2 } from '@grafana/data'; import { selectors } from '@grafana/e2e-selectors'; -import { config } from '@grafana/runtime'; import { InlineField, InlineSwitch, Input, Badge, useStyles2 } from '@grafana/ui'; export interface Props { @@ -11,22 +10,12 @@ export interface Props { isDefault: boolean; onNameChange: (name: string) => void; onDefaultChange: (value: boolean) => void; - alertingSupported: boolean; disabled?: boolean; } -export function BasicSettings({ - dataSourceName, - isDefault, - onDefaultChange, - onNameChange, - alertingSupported, - disabled, -}: Props) { +export function BasicSettings({ dataSourceName, isDefault, onDefaultChange, onNameChange, disabled }: Props) { return ( <> - {!config.featureToggles.dataSourcePageHeader && } -
{/* Name */} diff --git a/public/app/features/datasources/components/EditDataSource.tsx b/public/app/features/datasources/components/EditDataSource.tsx index a837c1ec030..a0104d828f7 100644 --- a/public/app/features/datasources/components/EditDataSource.tsx +++ b/public/app/features/datasources/components/EditDataSource.tsx @@ -120,10 +120,6 @@ export function EditDataSourceView({ const dsi = getDataSourceSrv()?.getInstanceSettings(dataSource.uid); - const hasAlertingEnabled = Boolean(dsi?.meta?.alerting ?? false); - const isAlertManagerDatasource = dsi?.type === 'alertmanager'; - const alertingSupported = hasAlertingEnabled || isAlertManagerDatasource; - const onSubmit = async (e: React.MouseEvent | React.FormEvent) => { e.preventDefault(); trackDsConfigClicked('save_and_test'); @@ -190,7 +186,6 @@ export function EditDataSourceView({ isDefault={dataSource.isDefault} onDefaultChange={onDefaultChange} onNameChange={onNameChange} - alertingSupported={alertingSupported} disabled={readOnly || !hasWriteRights} /> diff --git a/public/app/features/datasources/state/navModel.ts b/public/app/features/datasources/state/navModel.ts index c68e547b7f0..9223eaa2e82 100644 --- a/public/app/features/datasources/state/navModel.ts +++ b/public/app/features/datasources/state/navModel.ts @@ -13,7 +13,6 @@ const loadingDSType = 'Loading'; export function buildNavModel(dataSource: DataSourceSettings, plugin: GenericDataSourcePlugin): NavModelItem { const pluginMeta = plugin.meta; const highlightsEnabled = config.featureToggles.featureHighlights; - const dataSourcePageHeader = config.featureToggles.dataSourcePageHeader; const navModel: NavModelItem = { img: pluginMeta.info.logos.large, id: 'datasource-' + dataSource.uid, @@ -30,10 +29,6 @@ export function buildNavModel(dataSource: DataSourceSettings, plugin: GenericDat ], }; - if (!dataSourcePageHeader) { - navModel.subTitle = `Type: ${pluginMeta.name}`; - } - if (plugin.configPages) { for (const page of plugin.configPages) { navModel.children!.push({