fix(group-by): fixing error while changing the group by from a key to none (#1453)

itssharmasandeep · web-flow · commit 46c423d9a583 · 2022-02-23T21:59:00.000+05:30
diff --git a/projects/observability/src/shared/components/explore-query-editor/group-by/explore-query-group-by-editor.component.ts b/projects/observability/src/shared/components/explore-query-editor/group-by/explore-query-group-by-editor.component.ts
@@ -83,7 +83,7 @@ export class ExploreQueryGroupByEditorComponent implements OnChanges {
       .pipe(
         filter(expression => this.isValidExpressionToEmit(expression)),
         debounceTime(500),
-        map(expression => omit(expression, 'metadata'))
+        map(expression => (isEmpty(expression) ? undefined : omit(expression, 'metadata')))
       )
       .subscribe(this.groupByExpressionChange);
   }

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ export class ExploreQueryGroupByEditorComponent implements OnChanges {`
`83`	`83`	`.pipe(`
`84`	`84`	`filter(expression => this.isValidExpressionToEmit(expression)),`
`85`	`85`	`debounceTime(500),`
`86`		`- map(expression => omit(expression, 'metadata'))`
	`86`	`+ map(expression => (isEmpty(expression) ? undefined : omit(expression, 'metadata')))`
`87`	`87`	`)`
`88`	`88`	`.subscribe(this.groupByExpressionChange);`
`89`	`89`	`}`