Browse Source

fix(onboarding): Add wizard URL param to sourcemaps step (#77840)

ArthurKnaus 5 months ago
parent
commit
29ca132f84

+ 6 - 8
static/app/components/onboarding/gettingStartedDoc/utils/index.tsx

@@ -1,8 +1,7 @@
 import Alert from 'sentry/components/alert';
 import ExternalLink from 'sentry/components/links/externalLink';
+import type {DocsParams} from 'sentry/components/onboarding/gettingStartedDoc/types';
 import {t, tct} from 'sentry/locale';
-import type {Organization} from 'sentry/types/organization';
-import type {PlatformKey} from 'sentry/types/project';
 import {trackAnalytics} from 'sentry/utils/analytics';
 
 export function getUploadSourceMapsStep({
@@ -11,13 +10,12 @@ export function getUploadSourceMapsStep({
   platformKey,
   projectId,
   newOrg,
-}: {
+  isSelfHosted,
+  urlPrefix,
+}: DocsParams & {
   guideLink: string;
-  newOrg?: boolean;
-  organization?: Organization;
-  platformKey?: PlatformKey;
-  projectId?: string;
 }) {
+  const urlParam = !isSelfHosted && urlPrefix ? `--url ${urlPrefix}` : '';
   return {
     title: t('Upload Source Maps'),
     description: (
@@ -33,7 +31,7 @@ export function getUploadSourceMapsStep({
     configurations: [
       {
         language: 'bash',
-        code: `npx @sentry/wizard@latest -i sourcemaps`,
+        code: `npx @sentry/wizard@latest -i sourcemaps ${urlParam}`,
         onCopy: () => {
           if (!organization || !projectId || !platformKey) {
             return;

+ 1 - 0
static/app/gettingStartedDocs/javascript/ember.tsx

@@ -145,6 +145,7 @@ const onboarding: OnboardingConfig = {
     },
     getUploadSourceMapsStep({
       guideLink: 'https://docs.sentry.io/platforms/javascript/guides/ember/sourcemaps/',
+      ...params,
     }),
   ],
   verify: () => [

+ 1 - 0
static/app/gettingStartedDocs/javascript/gatsby.tsx

@@ -177,6 +177,7 @@ const onboarding: OnboardingConfig = {
     getConfigureStep(params),
     getUploadSourceMapsStep({
       guideLink: 'https://docs.sentry.io/platforms/javascript/guides/gatsby/sourcemaps//',
+      ...params,
     }),
   ],
   verify: () => [

+ 1 - 0
static/app/gettingStartedDocs/javascript/javascript.tsx

@@ -147,6 +147,7 @@ const onboarding: OnboardingConfig = {
     },
     getUploadSourceMapsStep({
       guideLink: 'https://docs.sentry.io/platforms/javascript/sourcemaps/',
+      ...params,
     }),
   ],
   verify: () => [

+ 1 - 0
static/app/gettingStartedDocs/javascript/react.tsx

@@ -153,6 +153,7 @@ const onboarding: OnboardingConfig = {
     },
     getUploadSourceMapsStep({
       guideLink: 'https://docs.sentry.io/platforms/javascript/guides/react/sourcemaps/',
+      ...params,
     }),
   ],
   verify: () => [

+ 1 - 0
static/app/gettingStartedDocs/javascript/solid.tsx

@@ -165,6 +165,7 @@ const onboarding: OnboardingConfig = {
     },
     getUploadSourceMapsStep({
       guideLink: 'https://docs.sentry.io/platforms/javascript/guides/solid/sourcemaps/',
+      ...params,
     }),
   ],
   verify: () => [

+ 1 - 0
static/app/gettingStartedDocs/javascript/svelte.tsx

@@ -162,6 +162,7 @@ const onboarding: OnboardingConfig = {
     },
     getUploadSourceMapsStep({
       guideLink: 'https://docs.sentry.io/platforms/javascript/guides/svelte/sourcemaps/',
+      ...params,
     }),
   ],
   verify: () => [

+ 1 - 0
static/app/gettingStartedDocs/node/node.tsx

@@ -90,6 +90,7 @@ const onboarding: OnboardingConfig = {
     },
     getUploadSourceMapsStep({
       guideLink: 'https://docs.sentry.io/platforms/javascript/guides/node/sourcemaps/',
+      ...params,
     }),
   ],
   verify: ({isPerformanceSelected}) => [