Skip to content

Commit 72da70d

Browse files
authored
Merge pull request #1239 from scottsut/fix
fix: wrong logic on chart config change
2 parents 2a44bd5 + 3aaf86b commit 72da70d

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

frontend/src/app/components/ChartEditor.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -339,7 +339,11 @@ export const ChartEditor: FC<ChartEditorProps> = ({
339339
setAllowQuery(payload.needRefresh);
340340
return true;
341341
}
342-
if (payload.value.type !== ChartDataSectionType.FILTER) {
342+
// generate runtime computed fields(date level)
343+
if (
344+
payload.value.type === ChartDataSectionType.GROUP ||
345+
payload.value.type === ChartDataSectionType.MIXED
346+
) {
343347
const dateLevelComputedFields = payload.value.rows.filter(
344348
v => v.category === ChartDataViewFieldCategory.DateLevelComputedField,
345349
);

0 commit comments

Comments
 (0)