Skip to content

Commit c51c21b

Browse files
♻️ use constant
1 parent 5633d73 commit c51c21b

10 files changed

+21
-10
lines changed

static/app/views/insights/sessions/charts/chartWithIssues.tsx

+3-2
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ import type {DiscoverSeries} from 'sentry/views/insights/common/queries/useDisco
2121
import {ModalChartContainer} from 'sentry/views/insights/pages/backend/laravel/styles';
2222
import {WidgetVisualizationStates} from 'sentry/views/insights/pages/backend/laravel/widgetVisualizationStates';
2323
import useRecentIssues from 'sentry/views/insights/sessions/queries/useRecentIssues';
24+
import {SESSION_HEALTH_CHART_HEIGHT} from 'sentry/views/insights/sessions/utils/sessions';
2425

2526
export default function ChartWithIssues({
2627
project,
@@ -59,7 +60,7 @@ export default function ChartWithIssues({
5960
if (isLoading) {
6061
return (
6162
<Widget
62-
height={400}
63+
height={SESSION_HEALTH_CHART_HEIGHT}
6364
Visualization={<TimeSeriesWidgetVisualization.LoadingPlaceholder />}
6465
/>
6566
);
@@ -94,7 +95,7 @@ export default function ChartWithIssues({
9495
return (
9596
<Widget
9697
Title={<Widget.WidgetTitle title={title} />}
97-
height={400}
98+
height={SESSION_HEALTH_CHART_HEIGHT}
9899
Visualization={visualization}
99100
Actions={
100101
<Widget.WidgetToolbar>

static/app/views/insights/sessions/charts/crashFreeSessionsChart.tsx

+2-1
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ import {t, tct} from 'sentry/locale';
33
import {formatSeriesName} from 'sentry/views/dashboards/widgets/timeSeriesWidget/formatters/formatSeriesName';
44
import {InsightsLineChartWidget} from 'sentry/views/insights/common/components/insightsLineChartWidget';
55
import useCrashFreeSessions from 'sentry/views/insights/sessions/queries/useCrashFreeSessions';
6+
import {SESSION_HEALTH_CHART_HEIGHT} from 'sentry/views/insights/sessions/utils/sessions';
67

78
export default function CrashFreeSessionsChart() {
89
const {series, releases, isPending, error} = useCrashFreeSessions();
@@ -17,7 +18,7 @@ export default function CrashFreeSessionsChart() {
1718
return (
1819
<InsightsLineChartWidget
1920
title={t('Crash Free Sessions')}
20-
height={400}
21+
height={SESSION_HEALTH_CHART_HEIGHT}
2122
description={tct(
2223
'The percent of sessions terminating without a crash. See [link:session status].',
2324
{

static/app/views/insights/sessions/charts/errorFreeSessionsChart.tsx

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import ExternalLink from 'sentry/components/links/externalLink';
22
import {t, tct} from 'sentry/locale';
33
import {InsightsLineChartWidget} from 'sentry/views/insights/common/components/insightsLineChartWidget';
44
import useErrorFreeSessions from 'sentry/views/insights/sessions/queries/useErrorFreeSessions';
5+
import {SESSION_HEALTH_CHART_HEIGHT} from 'sentry/views/insights/sessions/utils/sessions';
56

67
export default function ErrorFreeSessionsChart() {
78
const {series, isPending, error} = useErrorFreeSessions();
@@ -13,7 +14,7 @@ export default function ErrorFreeSessionsChart() {
1314
return (
1415
<InsightsLineChartWidget
1516
title={t('Error Free Sessions')}
16-
height={400}
17+
height={SESSION_HEALTH_CHART_HEIGHT}
1718
description={tct(
1819
'The percent of sessions terminating without a single error occurring. See [link:session status].',
1920
{

static/app/views/insights/sessions/charts/releaseSessionCountChart.tsx

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import {t} from 'sentry/locale';
22
import {formatSeriesName} from 'sentry/views/dashboards/widgets/timeSeriesWidget/formatters/formatSeriesName';
33
import {InsightsLineChartWidget} from 'sentry/views/insights/common/components/insightsLineChartWidget';
44
import useReleaseSessionCounts from 'sentry/views/insights/sessions/queries/useReleaseSessionCounts';
5+
import {SESSION_HEALTH_CHART_HEIGHT} from 'sentry/views/insights/sessions/utils/sessions';
56

67
export default function ReleaseSessionCountChart() {
78
const {series, releases, isPending, error} = useReleaseSessionCounts();
@@ -14,7 +15,7 @@ export default function ReleaseSessionCountChart() {
1415
return (
1516
<InsightsLineChartWidget
1617
title={t('Total Sessions by Release')}
17-
height={400}
18+
height={SESSION_HEALTH_CHART_HEIGHT}
1819
description={t(
1920
'The total number of sessions per release. The 5 most recent releases are shown.'
2021
)}

static/app/views/insights/sessions/charts/releaseSessionPercentageChart.tsx

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import {t} from 'sentry/locale';
22
import {formatSeriesName} from 'sentry/views/dashboards/widgets/timeSeriesWidget/formatters/formatSeriesName';
33
import {InsightsAreaChartWidget} from 'sentry/views/insights/common/components/insightsAreaChartWidget';
44
import useReleaseSessionPercentage from 'sentry/views/insights/sessions/queries/useReleaseSessionPercentage';
5+
import {SESSION_HEALTH_CHART_HEIGHT} from 'sentry/views/insights/sessions/utils/sessions';
56

67
export default function ReleaseSessionPercentageChart() {
78
const {series, releases, isPending, error} = useReleaseSessionPercentage();
@@ -14,7 +15,7 @@ export default function ReleaseSessionPercentageChart() {
1415
return (
1516
<InsightsAreaChartWidget
1617
title={t('Release Adoption')}
17-
height={400}
18+
height={SESSION_HEALTH_CHART_HEIGHT}
1819
description={t(
1920
'The percentage of total sessions that each release accounted for. The 5 most recent releases are shown.'
2021
)}

static/app/views/insights/sessions/charts/sessionHealthCountChart.tsx

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import ExternalLink from 'sentry/components/links/externalLink';
22
import {t, tct} from 'sentry/locale';
33
import {InsightsLineChartWidget} from 'sentry/views/insights/common/components/insightsLineChartWidget';
44
import useSessionHealthBreakdown from 'sentry/views/insights/sessions/queries/useSessionHealthBreakdown';
5+
import {SESSION_HEALTH_CHART_HEIGHT} from 'sentry/views/insights/sessions/utils/sessions';
56

67
export default function SessionHealthCountChart() {
78
const {series, isPending, error} = useSessionHealthBreakdown({type: 'count'});
@@ -15,7 +16,7 @@ export default function SessionHealthCountChart() {
1516

1617
return (
1718
<InsightsLineChartWidget
18-
height={400}
19+
height={SESSION_HEALTH_CHART_HEIGHT}
1920
title={t('Session Counts')}
2021
description={tct(
2122
'The count of sessions with each health status. See [link:session status].',

static/app/views/insights/sessions/charts/sessionHealthRateChart.tsx

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import ExternalLink from 'sentry/components/links/externalLink';
22
import {t, tct} from 'sentry/locale';
33
import {InsightsAreaChartWidget} from 'sentry/views/insights/common/components/insightsAreaChartWidget';
44
import useSessionHealthBreakdown from 'sentry/views/insights/sessions/queries/useSessionHealthBreakdown';
5+
import {SESSION_HEALTH_CHART_HEIGHT} from 'sentry/views/insights/sessions/utils/sessions';
56

67
export default function SessionHealthRateChart() {
78
const {series, isPending, error} = useSessionHealthBreakdown({type: 'rate'});
@@ -16,7 +17,7 @@ export default function SessionHealthRateChart() {
1617
return (
1718
<InsightsAreaChartWidget
1819
title={t('Session Health')}
19-
height={400}
20+
height={SESSION_HEALTH_CHART_HEIGHT}
2021
description={tct(
2122
'The percent of sessions with each health status. See [link:session status].',
2223
{

static/app/views/insights/sessions/charts/userHealthCountChart.tsx

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import ExternalLink from 'sentry/components/links/externalLink';
22
import {t, tct} from 'sentry/locale';
33
import {InsightsLineChartWidget} from 'sentry/views/insights/common/components/insightsLineChartWidget';
44
import useUserHealthBreakdown from 'sentry/views/insights/sessions/queries/useUserHealthBreakdown';
5+
import {SESSION_HEALTH_CHART_HEIGHT} from 'sentry/views/insights/sessions/utils/sessions';
56

67
export default function UserHealthCountChart() {
78
const {series, isPending, error} = useUserHealthBreakdown({type: 'count'});
@@ -15,7 +16,7 @@ export default function UserHealthCountChart() {
1516

1617
return (
1718
<InsightsLineChartWidget
18-
height={400}
19+
height={SESSION_HEALTH_CHART_HEIGHT}
1920
title={t('User Counts')}
2021
description={tct(
2122
'Breakdown of total [linkUsers:users], grouped by [linkStatus:health status].',

static/app/views/insights/sessions/charts/userHealthRateChart.tsx

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import ExternalLink from 'sentry/components/links/externalLink';
22
import {t, tct} from 'sentry/locale';
33
import {InsightsAreaChartWidget} from 'sentry/views/insights/common/components/insightsAreaChartWidget';
44
import useUserHealthBreakdown from 'sentry/views/insights/sessions/queries/useUserHealthBreakdown';
5+
import {SESSION_HEALTH_CHART_HEIGHT} from 'sentry/views/insights/sessions/utils/sessions';
56

67
export default function UserHealthRateChart() {
78
const {series, isPending, error} = useUserHealthBreakdown({type: 'rate'});
@@ -16,7 +17,7 @@ export default function UserHealthRateChart() {
1617
return (
1718
<InsightsAreaChartWidget
1819
title={t('User Health')}
19-
height={400}
20+
height={SESSION_HEALTH_CHART_HEIGHT}
2021
description={tct(
2122
'The percent of [linkUsers:users] with each [linkStatus:health status].',
2223
{

static/app/views/insights/sessions/utils/sessions.tsx

+2
Original file line numberDiff line numberDiff line change
@@ -14,3 +14,5 @@ export const getCountStatusSeries = (
1414
groups.find(group => group.by['session.status'] === status)?.series[
1515
'count_unique(user)'
1616
] ?? [];
17+
18+
export const SESSION_HEALTH_CHART_HEIGHT = 400;

0 commit comments

Comments
 (0)