Skip to content

Commit d942833

Browse files
committed
wip
1 parent 5f0f474 commit d942833

File tree

3 files changed

+14
-9
lines changed

3 files changed

+14
-9
lines changed

static/app/views/insights/http/components/tables/domainsTable.tsx

+5-4
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import {renderHeadCell} from 'sentry/views/insights/common/components/tableCells
1919
import {QueryParameterNames} from 'sentry/views/insights/common/views/queryParameters';
2020
import {DataTitles} from 'sentry/views/insights/common/views/spans/types';
2121
import {DomainCell} from 'sentry/views/insights/http/components/tables/domainCell';
22+
import {HTTP_MODULE_URL_FIELD} from 'sentry/views/insights/http/views/httpLandingPage';
2223
import {ModuleName, type SpanMetricsResponse} from 'sentry/views/insights/types';
2324

2425
type Row = Pick<
@@ -37,7 +38,7 @@ type Row = Pick<
3738

3839
type Column = GridColumnHeader<
3940
| 'span.domain'
40-
| 'url'
41+
| 'span.description'
4142
| 'project'
4243
| 'spm()'
4344
| 'http_response_rate(3)'
@@ -85,7 +86,7 @@ export function DomainsTable({response, showRoutes, sort}: Props) {
8586
...(showRoutes
8687
? [
8788
{
88-
key: 'url',
89+
key: HTTP_MODULE_URL_FIELD,
8990
name: t('URL'),
9091
width: COL_WIDTH_UNDEFINED,
9192
},
@@ -199,12 +200,12 @@ function renderBodyCell(
199200
);
200201
}
201202

202-
if (column.key === 'url') {
203+
if (column.key === HTTP_MODULE_URL_FIELD) {
203204
return (
204205
<DomainCell
205206
projectId={row['project.id']?.toString()}
206207
domain={row['span.domain']}
207-
url={row.url}
208+
url={row[HTTP_MODULE_URL_FIELD]}
208209
/>
209210
);
210211
}

static/app/views/insights/http/views/httpDomainSummaryPage.tsx

+7-4
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,10 @@ import {
5050
MODULE_DOC_LINK,
5151
NULL_DOMAIN_DESCRIPTION,
5252
} from 'sentry/views/insights/http/settings';
53-
import {USE_EAP_HTTP_MODULE} from 'sentry/views/insights/http/views/httpLandingPage';
53+
import {
54+
HTTP_MODULE_URL_FIELD,
55+
USE_EAP_HTTP_MODULE,
56+
} from 'sentry/views/insights/http/views/httpLandingPage';
5457
import {BackendHeader} from 'sentry/views/insights/pages/backend/backendPageHeader';
5558
import {BACKEND_LANDING_SUB_PATH} from 'sentry/views/insights/pages/backend/settings';
5659
import {FrontendHeader} from 'sentry/views/insights/pages/frontend/frontendPageHeader';
@@ -94,7 +97,7 @@ export function HTTPDomainSummaryPage() {
9497
const filters: SpanMetricsQueryFilters = {
9598
...BASE_FILTERS,
9699
...(url
97-
? {url}
100+
? {[HTTP_MODULE_URL_FIELD]: url}
98101
: {'span.domain': domain === '' ? EMPTY_OPTION_VALUE : escapeFilterValue(domain)}),
99102
...(subregions.length > 0
100103
? {
@@ -216,7 +219,7 @@ export function HTTPDomainSummaryPage() {
216219
headerTitle: (
217220
<Fragment>
218221
{project && <ProjectAvatar project={project} size={36} />}
219-
{domain || NULL_DOMAIN_DESCRIPTION}
222+
{domain || url || NULL_DOMAIN_DESCRIPTION}
220223
<DomainStatusLink domain={domain} />
221224
</Fragment>
222225
),
@@ -237,7 +240,7 @@ export function HTTPDomainSummaryPage() {
237240
<ModuleBodyUpsellHook moduleName={ModuleName.HTTP}>
238241
<Layout.Body>
239242
<Layout.Main fullWidth>
240-
{domain === '' && (
243+
{domain === '' && !url && (
241244
<Alert type="info">
242245
{tct(
243246
'"Unknown Domain" entries can be caused by instrumentation errors. Please refer to our [link] for more information.',

static/app/views/insights/http/views/httpLandingPage.tsx

+2-1
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@ import {useDomainViewFilters} from 'sentry/views/insights/pages/useFilters';
5050
import {ModuleName, SpanMetricsField} from 'sentry/views/insights/types';
5151

5252
export const USE_EAP_HTTP_MODULE = true;
53+
export const HTTP_MODULE_URL_FIELD = 'span.description';
5354

5455
export function HTTPLandingPage() {
5556
const organization = useOrganization();
@@ -178,7 +179,7 @@ export function HTTPLandingPage() {
178179
fields: [
179180
'project',
180181
'project.id',
181-
...(shouldShowRoutes ? ['url'] : ['span.domain']),
182+
...(shouldShowRoutes ? [HTTP_MODULE_URL_FIELD] : ['span.domain']),
182183
'spm()',
183184
'avg(span.self_time)',
184185
'sum(span.self_time)',

0 commit comments

Comments
 (0)