Skip to content

Commit e625053

Browse files
authored
fix(dashboards-eap): Pass along sampled flag for disabling message (#89528)
If the data is at 100% sampling, we don't want to show the low accuracy message. Builds off of #89202 to pass along the `isSampled` flag that we were already collecting.
1 parent d647985 commit e625053

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

static/app/views/dashboards/widgetCard/chart.tsx

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ type WidgetCardChartProps = Pick<
8585
confidence?: Confidence;
8686
expandNumbers?: boolean;
8787
isMobile?: boolean;
88+
isSampled?: boolean | null;
8889
legendOptions?: LegendComponentOption;
8990
minTableColumnWidth?: string;
9091
noPadding?: boolean;
@@ -281,6 +282,7 @@ class WidgetCardChart extends Component<WidgetCardChartProps> {
281282
confidence,
282283
showConfidenceWarning,
283284
sampleCount,
285+
isSampled,
284286
} = this.props;
285287

286288
if (errorMessage) {
@@ -552,6 +554,7 @@ class WidgetCardChart extends Component<WidgetCardChartProps> {
552554
confidence={confidence}
553555
sampleCount={sampleCount}
554556
topEvents={topEventsCountExcludingOther}
557+
isSampled={isSampled}
555558
/>
556559
)}
557560
</ChartWrapper>

static/app/views/dashboards/widgetCard/widgetCardChartContainer.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,7 @@ export function WidgetCardChartContainer({
113113
timeseriesResultsTypes,
114114
confidence,
115115
sampleCount,
116+
isSampled,
116117
}) => {
117118
if (widget.widgetType === WidgetType.ISSUE) {
118119
return (
@@ -172,6 +173,7 @@ export function WidgetCardChartContainer({
172173
confidence={confidence}
173174
sampleCount={sampleCount}
174175
minTableColumnWidth={minTableColumnWidth}
176+
isSampled={isSampled}
175177
/>
176178
</Fragment>
177179
);

0 commit comments

Comments
 (0)