QT/superset2
1
0
Fork 0
mirror of https://github.com/apache/superset.git synced 2026-04-19 16:14:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(DrillBy): make drill by work with multi metric charts (#34171)

Browse Source
This commit is contained in:
Mehmet Salih Yavuz
2025-07-15 16:25:48 +03:00
committed by GitHub
parent 67b21c45df
commit 31a15c5162
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/EchartsTimeseries.tsx
View File

@@ -209,12 +209,18 @@ export default function EchartsTimeseries({
}),
);
groupBy.forEach((dimension, i) => {
const val = labelMap[seriesName][i];
const dimensionValues = labelMap[seriesName] ?? [];
// Skip the metric values at the beginning and get the actual dimension value
// If we have multiple metrics, they come first, then the dimension values
const metricsCount = dimensionValues.length - groupBy.length;
const val = dimensionValues[metricsCount + i];
drillByFilters.push({
col: dimension,
op: '==',
val,
formattedVal: formatSeriesName(values[i], {
formattedVal: formatSeriesName(val, {
timeFormatter: getTimeFormatter(formData.dateFormat),
numberFormatter: getNumberFormatter(formData.numberFormat),
coltype: coltypeMapping?.[getColumnLabel(dimension)],