Skip to content

Commit

Permalink
rename screen_load and app_start in ModuleName
Browse files Browse the repository at this point in the history
  • Loading branch information
KevinL10 committed May 15, 2024
1 parent 168d3af commit db27a58
Show file tree
Hide file tree
Showing 7 changed files with 11 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,7 @@ export function ScreenSummary() {
...(deviceClass ? {[SpanMetricsField.DEVICE_CLASS]: deviceClass} : {}),
}}
groupId={spanGroup}
moduleName={ModuleName.STARTUP}
moduleName={ModuleName.APP_START}
transactionName={transactionName}
spanDescription={spanDescription}
spanOp={spanOp}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -211,7 +211,7 @@ function ScreenLoadSpans() {
{spanGroup && (
<SpanSamplesPanel
groupId={spanGroup}
moduleName={ModuleName.SCREEN}
moduleName={ModuleName.SCREEN_LOAD}
transactionName={transactionName}
spanDescription={spanDescription}
onClose={() => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ export function ScreenLoadSpansTable({
onClick={() =>
trackAnalytics('performance_views.sample_spans.opened', {
organization,
source: ModuleName.SCREEN,
source: ModuleName.SCREEN_LOAD,
})
}
to={`${pathname}?${qs.stringify(query)}`}
Expand Down
4 changes: 2 additions & 2 deletions static/app/views/starfish/types.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@ export enum ModuleName {
CACHE = 'cache',
VITAL = 'vital',
QUEUE = 'queue',
SCREEN = 'screen',
STARTUP = 'startup',
SCREEN_LOAD = 'screen_load',
APP_START = 'app_start',
RESOURCE = 'resource',
ALL = '',
OTHER = 'other',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -105,8 +105,8 @@ const LABEL_FOR_MODULE_NAME: {[key in ModuleName]: ReactNode} = {
cache: t('Action'),
vital: t('Action'),
queue: t('Action'),
screen: t('Action'),
startup: t('Action'),
screen_load: t('Action'),
app_start: t('Action'),
resource: t('Resource'),
other: t('Action'),
'': t('Action'),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -173,8 +173,8 @@ const LABEL_FOR_MODULE_NAME: {[key in ModuleName]: ReactNode} = {
cache: t('Domain'),
vital: t('Domain'),
queue: t('Domain'),
screen: t('Domain'),
startup: t('Domain'),
screen_load: t('Domain'),
app_start: t('Domain'),
resource: t('Resource'),
other: t('Domain'),
'': t('Domain'),
Expand Down
4 changes: 2 additions & 2 deletions static/app/views/starfish/views/spans/spanTimeCharts.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,8 @@ export function SpanTimeCharts({
[ModuleName.CACHE]: [],
[ModuleName.VITAL]: [],
[ModuleName.QUEUE]: [],
[ModuleName.SCREEN]: [],
[ModuleName.STARTUP]: [],
[ModuleName.SCREEN_LOAD]: [],
[ModuleName.APP_START]: [],
[ModuleName.RESOURCE]: features.includes(
'starfish-browser-resource-module-bundle-analysis'
)
Expand Down

0 comments on commit db27a58

Please sign in to comment.