Skip to content

Commit a328328

Browse files
committed
Fix validation
1 parent 4c97ac3 commit a328328

2 files changed

Lines changed: 5 additions & 6 deletions

File tree

src/plugins/vis_types/timeseries/public/convert_to_lens/lib/metrics/supported_metrics.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -85,28 +85,28 @@ export const SUPPORTED_METRICS: { [key: string]: AggOptions } = {
8585
name: 'overall_average',
8686
isFullReference: true,
8787
isFieldRequired: true,
88-
supportedPanelTypes: [PANEL_TYPES.TIMESERIES, PANEL_TYPES.TOP_N],
88+
supportedPanelTypes: [PANEL_TYPES.TIMESERIES],
8989
supportedTimeRangeModes: [TIME_RANGE_DATA_MODES.ENTIRE_TIME_RANGE],
9090
},
9191
max_bucket: {
9292
name: 'overall_max',
9393
isFullReference: true,
9494
isFieldRequired: true,
95-
supportedPanelTypes: [PANEL_TYPES.TIMESERIES, PANEL_TYPES.TOP_N],
95+
supportedPanelTypes: [PANEL_TYPES.TIMESERIES],
9696
supportedTimeRangeModes: [TIME_RANGE_DATA_MODES.ENTIRE_TIME_RANGE],
9797
},
9898
min_bucket: {
9999
name: 'overall_min',
100100
isFullReference: true,
101101
isFieldRequired: true,
102-
supportedPanelTypes: [PANEL_TYPES.TIMESERIES, PANEL_TYPES.TOP_N],
102+
supportedPanelTypes: [PANEL_TYPES.TIMESERIES],
103103
supportedTimeRangeModes: [TIME_RANGE_DATA_MODES.ENTIRE_TIME_RANGE],
104104
},
105105
sum_bucket: {
106106
name: 'overall_sum',
107107
isFullReference: true,
108108
isFieldRequired: true,
109-
supportedPanelTypes: [PANEL_TYPES.TIMESERIES, PANEL_TYPES.TOP_N],
109+
supportedPanelTypes: [PANEL_TYPES.TIMESERIES],
110110
supportedTimeRangeModes: [TIME_RANGE_DATA_MODES.ENTIRE_TIME_RANGE],
111111
},
112112
max: {

src/plugins/vis_types/timeseries/public/convert_to_lens/lib/metrics/validate_metrics.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,7 @@ export const isValidMetrics = (metrics: Metric[], panelType: string, timeRangeMo
3434
return metrics.every((metric) => {
3535
const isMetricAggValid =
3636
metric.type !== 'filter_ratio' ||
37-
(metric.metric_agg &&
38-
isMetricValid(metric.metric_agg, panelType, metric.field, timeRangeMode));
37+
isMetricValid(metric.metric_agg || 'count', panelType, metric.field, timeRangeMode);
3938
return (
4039
metric.type === 'series_agg' ||
4140
(isMetricValid(metric.type, panelType, metric.field, timeRangeMode) && isMetricAggValid)

0 commit comments

Comments
 (0)