Skip to content

Commit 02695f9

Browse files
authored
ref(issues): Remove redundant Type row from Event Grouping Information table (#93892)
<!-- Describe your PR here. --> [ticket](https://linear.app/getsentry/issue/ID-156/grouping-info-remove-type-field-from-ui) The Type field in the Grouping Info section of the issue details page was redundant. This removes the Type row from all variant types while keeping the underlying data structure intact. before ![Screenshot 2025-06-20 at 12 00 54 PM](https://github.com/user-attachments/assets/97ca72da-0a52-4446-9825-cd4fcb505adf) after ![Screenshot 2025-06-20 at 11 59 29 AM](https://github.com/user-attachments/assets/a4284d2b-c9f5-442f-b010-7fe72a598e39)
1 parent b4d3000 commit 02695f9

File tree

1 file changed

+1
-63
lines changed

1 file changed

+1
-63
lines changed

static/app/components/events/groupingInfo/groupingVariant.tsx

Lines changed: 1 addition & 63 deletions
Original file line numberDiff line numberDiff line change
@@ -100,68 +100,19 @@ function GroupingVariant({event, showGroupingConfig, variant}: GroupingVariantPr
100100
switch (variant.type) {
101101
case EventGroupVariantType.COMPONENT:
102102
component = variant.component;
103-
data.push([
104-
t('Type'),
105-
<TextWithQuestionTooltip key="type">
106-
{variant.type}
107-
<QuestionTooltip
108-
size="xs"
109-
position="top"
110-
title={t(
111-
'Uses a complex grouping algorithm taking event data into account'
112-
)}
113-
/>
114-
</TextWithQuestionTooltip>,
115-
]);
103+
116104
if (showGroupingConfig && variant.config?.id) {
117105
data.push([t('Grouping Config'), variant.config.id]);
118106
}
119107
break;
120108
case EventGroupVariantType.CUSTOM_FINGERPRINT:
121-
data.push([
122-
t('Type'),
123-
<TextWithQuestionTooltip key="type">
124-
{variant.type}
125-
<QuestionTooltip
126-
size="xs"
127-
position="top"
128-
title={t('Overrides the default grouping by a custom fingerprinting rule')}
129-
/>
130-
</TextWithQuestionTooltip>,
131-
]);
132109
addFingerprintInfo(data, variant);
133110
break;
134111
case EventGroupVariantType.BUILT_IN_FINGERPRINT:
135-
data.push([
136-
t('Type'),
137-
<TextWithQuestionTooltip key="type">
138-
{variant.type}
139-
<QuestionTooltip
140-
size="xs"
141-
position="top"
142-
title={t(
143-
'Overrides the default grouping by a Sentry defined fingerprinting rule'
144-
)}
145-
/>
146-
</TextWithQuestionTooltip>,
147-
]);
148112
addFingerprintInfo(data, variant);
149113
break;
150114
case EventGroupVariantType.SALTED_COMPONENT:
151115
component = variant.component;
152-
data.push([
153-
t('Type'),
154-
<TextWithQuestionTooltip key="type">
155-
{variant.type}
156-
<QuestionTooltip
157-
size="xs"
158-
position="top"
159-
title={t(
160-
'Uses a complex grouping algorithm taking event data and a fingerprint into account'
161-
)}
162-
/>
163-
</TextWithQuestionTooltip>,
164-
]);
165116
addFingerprintInfo(data, variant);
166117
if (showGroupingConfig && variant.config?.id) {
167118
data.push([t('Grouping Config'), variant.config.id]);
@@ -173,19 +124,6 @@ function GroupingVariant({event, showGroupingConfig, variant}: GroupingVariantPr
173124
.find((c): c is EntrySpans => c.type === 'spans')
174125
?.data?.map((span: RawSpanType) => [span.span_id, span.hash]) ?? []
175126
);
176-
data.push([
177-
t('Type'),
178-
<TextWithQuestionTooltip key="type">
179-
{variant.type}
180-
<QuestionTooltip
181-
size="xs"
182-
position="top"
183-
title={t(
184-
'Uses the evidence from performance issue detection to generate a fingerprint.'
185-
)}
186-
/>
187-
</TextWithQuestionTooltip>,
188-
]);
189127

190128
data.push(['Performance Issue Type', variant.key]);
191129
data.push(['Span Operation', variant.evidence.op]);

0 commit comments

Comments
 (0)