Skip to content

Commit

Permalink
fix bug, pass date/groupby/metric column to dimension modal
Browse files Browse the repository at this point in the history
  • Loading branch information
liuyl committed Sep 4, 2023
1 parent 84a7b40 commit 53951e0
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 0 deletions.
3 changes: 3 additions & 0 deletions frontend/src/components/main-dashboard/MainDashboard.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -408,6 +408,9 @@ export default function MainDashboard() {
fileId={fileId}
targetDirection={targetDirection}
selectedColumns={selectedColumns}
dateColumn={dateColumn}
groupByColumns={groupByColumns}
metricColumn={metricColumn}
baseDateRange={baseDateRange}
comparisonDateRange={comparisonDateRange}
dataSourceType={dataSourceType}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ import { DataSourceType } from "../../../types/data-source";
import {
ColumnConfig,
DateRangeConfig,
MetricColumn,
TargetDirection,
} from "../../../types/report-config";
import { MetricOverviewTable } from "../MetricOverviewTable";
Expand All @@ -39,6 +40,9 @@ interface Props {
selectedColumns: {
[key: string]: ColumnConfig;
};
dateColumn: string;
metricColumn: MetricColumn,
groupByColumns: string[];
baseDateRange: DateRangeConfig;
comparisonDateRange: DateRangeConfig;
dataSourceType: DataSourceType;
Expand All @@ -48,6 +52,9 @@ export function DimensionSliceDetailModal({
targetDirection,
fileId,
selectedColumns,
dateColumn,
groupByColumns,
metricColumn,
baseDateRange,
comparisonDateRange,
dataSourceType,
Expand Down Expand Up @@ -76,6 +83,9 @@ export function DimensionSliceDetailModal({
body: JSON.stringify({
fileId,
baseDateRange,
dateColumn,
groupByColumns,
metricColumn,
comparisonDateRange,
selectedColumns,
segmentKey: selectedSliceKey,
Expand Down

0 comments on commit 53951e0

Please sign in to comment.