Skip to content

Commit ed3bd8f

Browse files
authored
fix: some very minor clean up (#1226)
* fix: some very minor clean up * style: prettier
1 parent 47957d7 commit ed3bd8f

File tree

2 files changed

+14
-10
lines changed

2 files changed

+14
-10
lines changed

projects/components/src/table/controls/table-controls.component.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,8 +102,6 @@ import {
102102
})
103103
export class TableControlsComponent implements OnChanges {
104104
public readonly DEFAULT_SEARCH_PLACEHOLDER: string = 'Search...';
105-
@Input()
106-
public persistenceId?: string;
107105

108106
@Input()
109107
public searchEnabled?: boolean;

projects/observability/src/shared/dashboard/widgets/table/table-widget-renderer.component.ts

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,19 @@ import { Renderer } from '@hypertrace/hyperdash';
3434
import { RendererApi, RENDERER_API } from '@hypertrace/hyperdash-angular';
3535
import { capitalize, isEmpty, isEqual, pick } from 'lodash-es';
3636
import { BehaviorSubject, combineLatest, Observable, of, Subject } from 'rxjs';
37-
import { filter, first, map, pairwise, share, startWith, switchMap, take, tap, withLatestFrom } from 'rxjs/operators';
37+
import {
38+
filter,
39+
first,
40+
map,
41+
mapTo,
42+
pairwise,
43+
share,
44+
startWith,
45+
switchMap,
46+
take,
47+
tap,
48+
withLatestFrom
49+
} from 'rxjs/operators';
3850
import { AttributeMetadata, toFilterAttributeType } from '../../../graphql/model/metadata/attribute-metadata';
3951
import { MetadataService } from '../../../services/metadata/metadata.service';
4052
import { InteractionHandler } from '../../interaction/interaction-handler';
@@ -286,13 +298,7 @@ export class TableWidgetRendererComponent
286298
private getColumnConfigs(): Observable<TableColumnConfig[]> {
287299
return this.getPreferences().pipe(
288300
switchMap(preferences =>
289-
combineLatest([
290-
this.getScope(),
291-
this.api.change$.pipe(
292-
map(() => true),
293-
startWith(true)
294-
)
295-
]).pipe(
301+
combineLatest([this.getScope(), this.api.change$.pipe(mapTo(true), startWith(true))]).pipe(
296302
switchMap(([scope]) => this.model.getColumns(scope)),
297303
startWith([]),
298304
map((columns: SpecificationBackedTableColumnDef[]) =>

0 commit comments

Comments
 (0)