Skip to content

Commit bc9e7f5

Browse files
authored
ref(flags): remove statsPeriod in flag details query (#89360)
The backend uses statsPeriod to clamp date range - imo we don't need this since we're already paginating.
1 parent d7a3924 commit bc9e7f5

File tree

1 file changed

+8
-13
lines changed

1 file changed

+8
-13
lines changed

static/app/views/issueDetails/groupFeatureFlags/flagDetailsDrawerContent.tsx

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import {Fragment, useMemo, useState} from 'react';
1+
import {Fragment, useState} from 'react';
22
import styled from '@emotion/styled';
33

44
import {useAnalyticsArea} from 'sentry/components/analyticsArea';
@@ -36,25 +36,20 @@ export function FlagDetailsDrawerContent() {
3636

3737
const sortArrow = <IconArrow color="gray300" size="xs" direction="down" />;
3838

39-
const flagQuery = useMemo(() => {
40-
return {
41-
flag: tagKey,
42-
per_page: 50,
43-
queryReferrer: 'featureFlagDetailsDrawer',
44-
statsPeriod: '90d',
45-
sort: '-created_at',
46-
cursor: location.query.flagDrawerCursor,
47-
};
48-
}, [tagKey, location.query.flagDrawerCursor]);
49-
5039
const {
5140
data: flagLog,
5241
isPending,
5342
isError,
5443
getResponseHeader,
5544
} = useOrganizationFlagLog({
5645
organization,
57-
query: flagQuery,
46+
query: {
47+
flag: tagKey,
48+
per_page: 50,
49+
queryReferrer: 'featureFlagDetailsDrawer',
50+
sort: '-created_at',
51+
cursor: location.query.flagDrawerCursor,
52+
},
5853
});
5954
const pageLinks = getResponseHeader?.('Link') ?? null;
6055

0 commit comments

Comments
 (0)