Skip to content

Commit 5078ea0

Browse files
AbhiPrasadonurtemizkan
authored andcommitted
fix(vue): Use ui category for span operations (#4220)
As per the new spec in https://develop.sentry.dev/sdk/performance/span-operations/#js-frameworks, we now want to prefix our vue spans operations with `ui`.
1 parent d5978f4 commit 5078ea0

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

packages/vue/src/tracing.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ import { formatComponentName } from './components';
66
import { DEFAULT_HOOKS } from './constants';
77
import { Hook, Operation, TracingOptions, ViewModel, Vue } from './types';
88

9+
const VUE_OP = 'ui.vue';
10+
911
type Mixins = Parameters<Vue['mixin']>[0];
1012

1113
interface VueSentry extends ViewModel {
@@ -75,7 +77,7 @@ export const createTracingMixins = (options: TracingOptions): Mixins => {
7577
this.$_sentryRootSpan ||
7678
activeTransaction.startChild({
7779
description: 'Application Render',
78-
op: 'vue',
80+
op: VUE_OP,
7981
});
8082
}
8183
}
@@ -105,7 +107,7 @@ export const createTracingMixins = (options: TracingOptions): Mixins => {
105107
if (activeTransaction) {
106108
this.$_sentrySpans[operation] = activeTransaction.startChild({
107109
description: `Vue <${name}>`,
108-
op: operation,
110+
op: `${VUE_OP}.${operation}`,
109111
});
110112
}
111113
}

0 commit comments

Comments
 (0)