Skip to content

Commit ac5436f

Browse files
scttcpercvxluo
authored andcommitted
feat(issues): Prefetch supergroup data and add stats columns to rows (#111719)
Eagerly fetches supergroup assignments in the issue list overview to block the stream from rendering until grouping data is available, preventing pop-in. Supergroup rows now show aggregated stats (last seen, age, trend chart, events, users) from member groups and align to the same column grid as regular issue rows. Removes the previous experiment. --------- Co-authored-by: Charlie Luo <cvxluo@gmail.com>
1 parent 08cf431 commit ac5436f

File tree

11 files changed

+488
-201
lines changed

11 files changed

+488
-201
lines changed

.github/CODEOWNERS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -678,6 +678,7 @@ tests/sentry/api/endpoints/test_organization_attribute_mappings.py @get
678678
/static/app/components/events/highlights/ @getsentry/issue-workflow
679679
/static/app/components/issues/ @getsentry/issue-workflow
680680
/static/app/components/stackTrace/ @getsentry/issue-workflow
681+
/static/app/components/stream/supergroupRow.tsx @getsentry/issue-detection-frontend
681682
/static/app/views/issueList/ @getsentry/issue-workflow
682683
/static/app/views/issueList/issueListSeerComboBox.tsx @getsentry/issue-workflow @getsentry/machine-learning-ai
683684
/static/app/views/issueList/pages/supergroups.tsx @getsentry/issue-detection-frontend

static/app/components/group/issueSuperGroup.tsx

Lines changed: 0 additions & 109 deletions
This file was deleted.

static/app/components/groupMetaRow.tsx

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ import {TimesTag} from 'sentry/components/group/inboxBadges/timesTag';
1313
import {UnhandledTag} from 'sentry/components/group/inboxBadges/unhandledTag';
1414
import {IssueReplayCount} from 'sentry/components/group/issueReplayCount';
1515
import {IssueSeerBadge} from 'sentry/components/group/issueSeerBadge';
16-
import {IssueSuperGroup} from 'sentry/components/group/issueSuperGroup';
1716
import ProjectBadge from 'sentry/components/idBadge/projectBadge';
1817
import {extractSelectionParameters} from 'sentry/components/pageFilters/parse';
1918
import {Placeholder} from 'sentry/components/placeholder';
@@ -24,7 +23,6 @@ import {defined} from 'sentry/utils';
2423
import {getTitle} from 'sentry/utils/events';
2524
import {useReplayCountForIssues} from 'sentry/utils/replayCount/useReplayCountForIssues';
2625
import {projectCanLinkToReplay} from 'sentry/utils/replays/projectSupportsReplay';
27-
import {useSuperGroupForIssues} from 'sentry/utils/supergroup/useSuperGroupForIssues';
2826
import {useLocation} from 'sentry/utils/useLocation';
2927
import {useOrganization} from 'sentry/utils/useOrganization';
3028

@@ -75,18 +73,13 @@ export function GroupMetaRow({data, showAssignee, showLifetime = true}: Props) {
7573

7674
const issuesPath = `/organizations/${organization.slug}/issues/`;
7775
const {getReplayCountForIssue} = useReplayCountForIssues();
78-
const {getSuperGroupForIssue} = useSuperGroupForIssues();
7976

8077
const showReplayCount =
8178
organization.features.includes('session-replay') &&
8279
projectCanLinkToReplay(organization, project) &&
8380
data.issueCategory &&
8481
!!getReplayCountForIssue(data.id, data.issueCategory);
8582

86-
const supergroup = organization.features.includes('top-issues-ui')
87-
? getSuperGroupForIssue(data.id)
88-
: undefined;
89-
9083
const autofixRunExists = getAutofixRunExists(data);
9184
const seerFixable = isIssueQuickFixable(data);
9285
const showSeer =
@@ -126,7 +119,6 @@ export function GroupMetaRow({data, showAssignee, showLifetime = true}: Props) {
126119
</CommentsLink>
127120
) : null,
128121
showReplayCount ? <IssueReplayCount group={data} /> : null,
129-
supergroup ? <IssueSuperGroup supergroup={supergroup} /> : null,
130122
showSeer ? <IssueSeerBadge group={data} key="issue-seer-badge" /> : null,
131123
logger ? (
132124
<LoggerAnnotation>

0 commit comments

Comments
 (0)