Skip to content

Commit 3dc316e

Browse files
authored
fix(profiling): Remove export button on continuous profiles (#96761)
This button doesnt work on continuous profiles, only on transaction style profiles.
1 parent 8ae3ad7 commit 3dc316e

File tree

1 file changed

+13
-10
lines changed

1 file changed

+13
-10
lines changed

static/app/components/profiling/flamegraph/flamegraphDrawer/flamegraphDrawer.tsx

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import {ExportProfileButton} from 'sentry/components/profiling/exportProfileButt
1010
import {IconPanel} from 'sentry/icons';
1111
import {t} from 'sentry/locale';
1212
import {space} from 'sentry/styles/space';
13+
import {defined} from 'sentry/utils';
1314
import type {
1415
CanvasPoolManager,
1516
CanvasScheduler,
@@ -206,16 +207,18 @@ const FlamegraphDrawer = memo(function FlamegraphDrawer(props: FlamegraphDrawerP
206207
: undefined
207208
}
208209
/>
209-
<ProfilingDetailsListItem margin="none">
210-
<ExportProfileButton
211-
size="zero"
212-
priority={theme.isChonk ? 'transparent' : undefined}
213-
eventId={params.eventId}
214-
projectId={params.projectId}
215-
orgId={orgSlug}
216-
disabled={params.eventId === undefined || params.projectId === undefined}
217-
/>
218-
</ProfilingDetailsListItem>
210+
{defined(params.eventId) && defined(params.projectId) && (
211+
<ProfilingDetailsListItem margin="none">
212+
<ExportProfileButton
213+
size="zero"
214+
priority={theme.isChonk ? 'transparent' : undefined}
215+
eventId={params.eventId}
216+
projectId={params.projectId}
217+
orgId={orgSlug}
218+
disabled={params.eventId === undefined || params.projectId === undefined}
219+
/>
220+
</ProfilingDetailsListItem>
221+
)}
219222
<Separator />
220223
<ProfilingDetailsListItem>
221224
<LayoutSelectionContainer>

0 commit comments

Comments
 (0)