Skip to content

Commit c6898e5

Browse files
refactor: addressing review comments
1 parent 093f57f commit c6898e5

File tree

4 files changed

+5
-5
lines changed

4 files changed

+5
-5
lines changed

projects/distributed-tracing/src/shared/dashboard/widgets/waterfall/waterfall/span-name/span-name-cell-data.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,5 @@ export interface SpanNameCellData {
33
protocolName: string;
44
name: string;
55
color?: string;
6-
error?: boolean;
6+
hasError?: boolean;
77
}

projects/distributed-tracing/src/shared/dashboard/widgets/waterfall/waterfall/span-name/span-name-table-cell-renderer.component.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ describe('Span name table cell renderer component', () => {
4545
const spanNameDataWithColor = {
4646
...spanNameData,
4747
color: 'blue',
48-
error: true
48+
hasError: true
4949
};
5050
const spectator = buildComponent({
5151
providers: [tableCellDataProvider(spanNameDataWithColor)]

projects/distributed-tracing/src/shared/dashboard/widgets/waterfall/waterfall/span-name/span-name-table-cell-renderer.component.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ import { WaterfallTableCellType } from './span-name-cell-type';
2222
<span class="text" data-sensitive-pii>{{ this.value.name }}</span>
2323
</div>
2424
<ht-icon
25-
*ngIf="this.value.error"
25+
*ngIf="this.value.hasError"
2626
class="error-icon"
2727
icon="${IconType.Error}"
2828
size="${IconSize.Medium}"

projects/distributed-tracing/src/shared/dashboard/widgets/waterfall/waterfall/waterfall-chart.service.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ export class WaterfallChartService {
4242
name: datum.name,
4343
serviceName: datum.serviceName,
4444
protocolName: datum.protocolName,
45-
error: datum.errorCount > 0
45+
hasError: datum.errorCount > 0
4646
},
4747
$$iconType: this.iconLookupService.forSpanType(datum.spanType)!,
4848
getChildren: () => of([]),
@@ -123,7 +123,7 @@ export class WaterfallChartService {
123123
const node = nodes.shift()!;
124124
let color;
125125

126-
if (node.$$spanName.error) {
126+
if (node.$$spanName.hasError) {
127127
// If span contains an error
128128
color = Color.Red5;
129129
} else if (colorMap.has(node.serviceName)) {

0 commit comments

Comments
 (0)