Skip to content

Commit c1ff82d

Browse files
Dan Fullerclaude
authored andcommitted
fix: Remove dead fromMetric/isMetricsData code paths after flag removal
The performance-mep-bannerless-ui flag graduation left dead code: - fromMetric query param handling in discover results - isMetricsData param threaded through getWidgetDiscoverUrl, getMenuOptions, and the query selector modal Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent c02d2da commit c1ff82d

File tree

7 files changed

+6
-39
lines changed

7 files changed

+6
-39
lines changed

static/app/components/modals/dashboardWidgetQuerySelectorModal.tsx

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ export type DashboardWidgetQuerySelectorModalOptions = {
2323
dashboardFilters: DashboardFilters | undefined;
2424
organization: Organization;
2525
widget: Widget;
26-
isMetricsData?: boolean;
2726
};
2827

2928
type Props = ModalRenderProps &
@@ -34,8 +33,7 @@ type Props = ModalRenderProps &
3433
};
3534

3635
function DashboardWidgetQuerySelectorModal(props: Props) {
37-
const {organization, widget, selection, isMetricsData, Body, Header, dashboardFilters} =
38-
props;
36+
const {organization, widget, selection, Body, Header, dashboardFilters} = props;
3937

4038
const renderQueries = () => {
4139
const querySearchBars = widget.queries.map((query, index) => {
@@ -46,9 +44,7 @@ function DashboardWidgetQuerySelectorModal(props: Props) {
4644
},
4745
dashboardFilters,
4846
selection,
49-
organization,
50-
0,
51-
isMetricsData
47+
organization
5248
);
5349
return (
5450
<Fragment key={index}>

static/app/components/modals/dataWidgetViewerModal.tsx

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -98,10 +98,7 @@ import {
9898
SESSION_DURATION_ALERT,
9999
WidgetDescription,
100100
} from 'sentry/views/dashboards/widgetCard';
101-
import {
102-
DashboardsMEPProvider,
103-
useDashboardsMEPContext,
104-
} from 'sentry/views/dashboards/widgetCard/dashboardsMEPContext';
101+
import {DashboardsMEPProvider} from 'sentry/views/dashboards/widgetCard/dashboardsMEPContext';
105102
import type {GenericWidgetQueriesResult} from 'sentry/views/dashboards/widgetCard/genericWidgetQueries';
106103
import {IssueWidgetQueries} from 'sentry/views/dashboards/widgetCard/issueWidgetQueries';
107104
import {ReleaseWidgetQueries} from 'sentry/views/dashboards/widgetCard/releaseWidgetQueries';
@@ -907,8 +904,6 @@ function OpenButton({
907904
}: OpenButtonProps) {
908905
let openLabel: string;
909906
let path: string;
910-
const {isMetricsData} = useDashboardsMEPContext();
911-
912907
switch (widget.widgetType) {
913908
case WidgetType.ISSUE:
914909
openLabel = t('Open in Issues');
@@ -940,9 +935,7 @@ function OpenButton({
940935
{...widget, queries: [widget.queries[selectedQueryIndex]!]},
941936
dashboardFilters,
942937
selection,
943-
organization,
944-
0,
945-
isMetricsData
938+
organization
946939
);
947940
break;
948941
}

static/app/views/dashboards/utils.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -209,8 +209,7 @@ export function getWidgetDiscoverUrl(
209209
dashboardFilters: DashboardFilters | undefined,
210210
selection: PageFilters,
211211
organization: Organization,
212-
index = 0,
213-
isMetricsData = false
212+
index = 0
214213
) {
215214
const eventView = eventViewFromWidget(widget.title, widget.queries[index]!, selection);
216215
const discoverLocation = eventView.getResultsViewUrlTarget(
@@ -272,10 +271,6 @@ export function getWidgetDiscoverUrl(
272271
dashboardFilters
273272
);
274273

275-
if (isMetricsData) {
276-
discoverLocation.query.fromMetric = 'true';
277-
}
278-
279274
// Pass empty string when projects is empty to preserve "My Projects" selection in URL
280275
const projectParam =
281276
selection.projects.length === 0 ? '' : discoverLocation.query.project;

static/app/views/dashboards/widgetCard/index.spec.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,6 @@ describe('Dashboards > WidgetCard', () => {
181181
await userEvent.click(await screen.findByLabelText('Widget actions'));
182182
await userEvent.click(screen.getByRole('menuitemradio', {name: 'Open in Discover'}));
183183
expect(spy).toHaveBeenCalledWith({
184-
isMetricsData: false,
185184
organization,
186185
widget: multipleQueryWidget,
187186
});

static/app/views/dashboards/widgetCard/index.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ import {Widget} from 'sentry/views/dashboards/widgets/widget/widget';
5555
import {useLLMContext} from 'sentry/views/seerExplorer/contexts/llmContext';
5656
import {registerLLMContext} from 'sentry/views/seerExplorer/contexts/registerLLMContext';
5757

58-
import {useDashboardsMEPContext} from './dashboardsMEPContext';
5958
import {VisualizationWidget} from './visualizationWidget';
6059
import {
6160
getMenuOptions,
@@ -223,7 +222,6 @@ function WidgetCard(props: Props) {
223222
query.aggregates.some(aggregate => aggregate.includes('session.duration'))
224223
);
225224

226-
const {isMetricsData} = useDashboardsMEPContext();
227225
const extractionStatus = useExtractionStatus({queryKey: widget});
228226
const onDemandWarning = useOnDemandWarning({widget});
229227
const transactionsDeprecationWarning = useTransactionsDeprecationWarning({
@@ -328,7 +326,6 @@ function WidgetCard(props: Props) {
328326
organization,
329327
selection,
330328
widget,
331-
Boolean(isMetricsData),
332329
props.widgetLimitReached,
333330
props.hasEditAccess,
334331
location,

static/app/views/dashboards/widgetCard/widgetCardContextMenu.tsx

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,6 @@ export function getMenuOptions(
167167
organization: Organization,
168168
selection: PageFilters,
169169
widget: Widget,
170-
isMetricsData: boolean,
171170
widgetLimitReached: boolean,
172171
hasEditAccess = true,
173172
location: Location,
@@ -198,9 +197,7 @@ export function getMenuOptions(
198197
widget,
199198
dashboardFilters,
200199
selection,
201-
organization,
202-
0,
203-
isMetricsData
200+
organization
204201
);
205202
menuOptions.push({
206203
key: 'open-in-discover',
@@ -230,7 +227,6 @@ export function getMenuOptions(
230227
openDashboardWidgetQuerySelectorModal({
231228
organization,
232229
widget,
233-
isMetricsData,
234230
dashboardFilters,
235231
});
236232
},

static/app/views/discover/results.tsx

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -178,15 +178,6 @@ export class Results extends Component<Props, State> {
178178

179179
componentDidMount() {
180180
const {organization, selection, location, isHomepage, navigate} = this.props;
181-
if (location.query.fromMetric) {
182-
navigate(
183-
{
184-
...location,
185-
query: {...location.query, fromMetric: undefined},
186-
},
187-
{replace: true}
188-
);
189-
}
190181
if (location.query[SHOW_UNPARAM_BANNER]) {
191182
this.setState({showUnparameterizedBanner: true});
192183
navigate(

0 commit comments

Comments
 (0)