Skip to content

Commit

Permalink
ref(flags): don't show feature flag section on any issues besides err…
Browse files Browse the repository at this point in the history
…ors (#82119)

the CTA was showing up for a [crons
issue](https://sentry.sentry.io/issues/6139435070/events/11af5c3a9188460ab5f2edd5b583a988/?project=1)
but we should probably hide it for that category.
  • Loading branch information
michellewzhang authored and evanh committed Dec 17, 2024
1 parent 52aedff commit 72ad136
Showing 1 changed file with 5 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ import {featureFlagOnboardingPlatforms} from 'sentry/data/platformCategories';
import {IconMegaphone, IconSearch} from 'sentry/icons';
import {t} from 'sentry/locale';
import type {Event, FeatureFlag} from 'sentry/types/event';
import type {Group} from 'sentry/types/group';
import {type Group, IssueCategory} from 'sentry/types/group';
import type {Project} from 'sentry/types/project';
import {trackAnalytics} from 'sentry/utils/analytics';
import {useFeedbackForm} from 'sentry/utils/useFeedbackForm';
Expand Down Expand Up @@ -197,6 +197,10 @@ export function EventFeatureFlagList({
}
}, [hasFlags, hydratedFlags.length, organization]);

if (group.issueCategory !== IssueCategory.ERROR) {
return null;
}

if (showCTA) {
return <FeatureFlagInlineCTA projectId={event.projectID} />;
}
Expand Down

0 comments on commit 72ad136

Please sign in to comment.