From a5d5f95d386660fdd3585f0d4908497dab8a4358 Mon Sep 17 00:00:00 2001 From: John Bodley <4567245+john-bodley@users.noreply.github.com> Date: Wed, 27 Oct 2021 21:34:09 -0700 Subject: [PATCH] chore: pass datasource to metrics control (#1438) * Update controlPanel.tsx * Update index.tsx * Update dndControls.tsx --- .../src/shared-controls/dndControls.tsx | 2 ++ .../superset-ui-chart-controls/src/shared-controls/index.tsx | 2 ++ .../superset-ui/plugins/plugin-chart-table/src/controlPanel.tsx | 1 + 3 files changed, 5 insertions(+) diff --git a/superset-frontend/temporary_superset_ui/superset-ui/packages/superset-ui-chart-controls/src/shared-controls/dndControls.tsx b/superset-frontend/temporary_superset_ui/superset-ui/packages/superset-ui-chart-controls/src/shared-controls/dndControls.tsx index a3aecaf89f..bbc67ab103 100644 --- a/superset-frontend/temporary_superset_ui/superset-ui/packages/superset-ui-chart-controls/src/shared-controls/dndControls.tsx +++ b/superset-frontend/temporary_superset_ui/superset-ui/packages/superset-ui-chart-controls/src/shared-controls/dndControls.tsx @@ -90,6 +90,7 @@ export const dnd_adhoc_metrics: SharedControlConfig<'DndMetricSelect'> = { mapStateToProps: ({ datasource }) => ({ columns: datasource ? datasource.columns : [], savedMetrics: datasource ? datasource.metrics : [], + datasource, datasourceType: datasource?.type, }), description: t('One or many metrics to display'), @@ -116,6 +117,7 @@ export const dnd_sort_by: SharedControlConfig<'DndMetricSelect'> = { mapStateToProps: ({ datasource }) => ({ columns: datasource?.columns || [], savedMetrics: datasource?.metrics || [], + datasource, datasourceType: datasource?.type, }), }; diff --git a/superset-frontend/temporary_superset_ui/superset-ui/packages/superset-ui-chart-controls/src/shared-controls/index.tsx b/superset-frontend/temporary_superset_ui/superset-ui/packages/superset-ui-chart-controls/src/shared-controls/index.tsx index 191953f2aa..a338a054df 100644 --- a/superset-frontend/temporary_superset_ui/superset-ui/packages/superset-ui-chart-controls/src/shared-controls/index.tsx +++ b/superset-frontend/temporary_superset_ui/superset-ui/packages/superset-ui-chart-controls/src/shared-controls/index.tsx @@ -134,6 +134,7 @@ const metrics: SharedControlConfig<'MetricsControl'> = { mapStateToProps: ({ datasource }) => ({ columns: datasource ? datasource.columns : [], savedMetrics: datasource ? datasource.metrics : [], + datasource, datasourceType: datasource?.type, }), description: t('One or many metrics to display'), @@ -362,6 +363,7 @@ const sort_by: SharedControlConfig<'MetricsControl'> = { mapStateToProps: ({ datasource }) => ({ columns: datasource?.columns || [], savedMetrics: datasource?.metrics || [], + datasource, datasourceType: datasource?.type, }), }; diff --git a/superset-frontend/temporary_superset_ui/superset-ui/plugins/plugin-chart-table/src/controlPanel.tsx b/superset-frontend/temporary_superset_ui/superset-ui/plugins/plugin-chart-table/src/controlPanel.tsx index 855e46ef36..c05e641c6a 100644 --- a/superset-frontend/temporary_superset_ui/superset-ui/plugins/plugin-chart-table/src/controlPanel.tsx +++ b/superset-frontend/temporary_superset_ui/superset-ui/plugins/plugin-chart-table/src/controlPanel.tsx @@ -146,6 +146,7 @@ const percent_metrics: typeof sharedControls.metrics = { mapStateToProps: ({ datasource, controls }, controlState) => ({ columns: datasource?.columns || [], savedMetrics: datasource?.metrics || [], + datasource, datasourceType: datasource?.type, queryMode: getQueryMode(controls), externalValidationErrors: validateAggControlValues(controls, [