Просмотр исходного кода

feat(onboarding): Remove unused next steps (#78461)

These next steps had inconsistent conditional logic based on the product
selection.
It was confusing and, at that point of onboarding, just a distraction.
We hooked up analytics, and the click rate was around 1%.
This PR removes the docs links for additional products and keeps
relevant information there (like integration to React Router).
## Before
![CleanShot 2024-10-02 at 13 31
12](https://github.com/user-attachments/assets/99b18b09-f37f-4a2a-847c-4d773c55e9d0)

## After
![CleanShot 2024-10-02 at 13 31
39](https://github.com/user-attachments/assets/2bf9107e-8821-42d0-90bf-ad486bb14b1e)

Closes https://github.com/getsentry/sentry/issues/78322
Matej Minar 5 месяцев назад
Родитель
Сommit
15fe55530b

+ 1 - 1
static/app/components/onboarding/gettingStartedDoc/onboardingLayout.tsx

@@ -170,7 +170,7 @@ export function OnboardingLayout({
         {nextSteps.length > 0 && (
           <Fragment>
             <Divider />
-            <h4>{t('Next Steps')}</h4>
+            <h4>{t('Additional Information')}</h4>
             <List symbol="bullet">
               {nextSteps
                 .filter((step): step is Exclude<typeof step, null> => step !== null)

+ 0 - 8
static/app/gettingStartedDocs/apple/ios.tsx

@@ -521,14 +521,6 @@ const onboarding: OnboardingConfig<PlatformOptions> = {
       description: t('Learn about our first class integration with SwiftUI.'),
       link: 'https://docs.sentry.io/platforms/apple/tracing/instrumentation/swiftui-instrumentation/',
     },
-    {
-      id: 'profiling',
-      name: t('Profiling'),
-      description: t(
-        'Collect and analyze performance profiles from real user devices in production.'
-      ),
-      link: 'https://docs.sentry.io/platforms/apple/profiling/',
-    },
   ],
 };
 

+ 0 - 8
static/app/gettingStartedDocs/apple/macos.tsx

@@ -215,14 +215,6 @@ const onboarding: OnboardingConfig = {
       description: t('Learn about our first class integration with SwiftUI.'),
       link: 'https://docs.sentry.io/platforms/apple/tracing/instrumentation/swiftui-instrumentation/',
     },
-    {
-      id: 'profiling',
-      name: t('Profiling'),
-      description: t(
-        'Collect and analyze performance profiles from real user devices in production.'
-      ),
-      link: 'https://docs.sentry.io/platforms/apple/profiling/',
-    },
   ],
 };
 

+ 0 - 3
static/app/gettingStartedDocs/bun/bun.spec.tsx

@@ -28,8 +28,5 @@ describe('bun onboarding docs', function () {
     expect(
       screen.queryByText(textWithMarkupMatcher(/tracesSampleRate: 1\.0,/))
     ).not.toBeInTheDocument();
-
-    // Renders next steps
-    expect(screen.getByRole('link', {name: 'Tracing'})).toBeInTheDocument();
   });
 });

+ 1 - 13
static/app/gettingStartedDocs/bun/bun.tsx

@@ -92,19 +92,7 @@ const onboarding: OnboardingConfig = {
       ],
     },
   ],
-  nextSteps: params =>
-    params.isPerformanceSelected
-      ? []
-      : [
-          {
-            id: 'performance-monitoring',
-            name: t('Tracing'),
-            description: t(
-              'Track down transactions to connect the dots between 10-second page loads and poor-performing API calls or slow database queries.'
-            ),
-            link: 'https://docs.sentry.io/platforms/javascript/guides/bun/tracing/',
-          },
-        ],
+  nextSteps: () => [],
 };
 
 const customMetricsOnboarding: OnboardingConfig = {

+ 1 - 21
static/app/gettingStartedDocs/capacitor/capacitor.tsx

@@ -242,7 +242,7 @@ const onboarding: OnboardingConfig<PlatformOptions> = {
       ],
     },
   ],
-  nextSteps: params => [
+  nextSteps: () => [
     {
       id: 'capacitor-android-setup',
       name: t('Capacitor 2 Setup'),
@@ -251,26 +251,6 @@ const onboarding: OnboardingConfig<PlatformOptions> = {
       ),
       link: 'https://docs.sentry.io/platforms/javascript/guides/capacitor/?#capacitor-2---android-specifics',
     },
-    params.isPerformanceSelected
-      ? null
-      : {
-          id: 'performance-monitoring',
-          name: t('Tracing'),
-          description: t(
-            'Track down transactions to connect the dots between 10-second page loads and poor-performing API calls or slow database queries.'
-          ),
-          link: 'https://docs.sentry.io/platforms/javascript/guides/capacitor/tracing/',
-        },
-    params.isReplaySelected
-      ? null
-      : {
-          id: 'session-replay',
-          name: t('Session Replay'),
-          description: t(
-            'Get to the root cause of an error or latency issue faster by seeing all the technical details related to that issue in one visual replay on your web application.'
-          ),
-          link: 'https://docs.sentry.io/platforms/javascript/guides/capacitor/session-replay/',
-        },
   ],
 };
 

+ 0 - 3
static/app/gettingStartedDocs/deno/deno.spec.tsx

@@ -28,8 +28,5 @@ describe('deno onboarding docs', function () {
     expect(
       screen.queryByText(textWithMarkupMatcher(/tracesSampleRate: 1\.0,/))
     ).not.toBeInTheDocument();
-
-    // Renders next steps
-    expect(screen.getByRole('link', {name: 'Tracing'})).toBeInTheDocument();
   });
 });

+ 1 - 13
static/app/gettingStartedDocs/deno/deno.tsx

@@ -101,19 +101,7 @@ const onboarding: OnboardingConfig = {
       ],
     },
   ],
-  nextSteps: params =>
-    params.isPerformanceSelected
-      ? []
-      : [
-          {
-            id: 'performance-monitoring',
-            name: t('Tracing'),
-            description: t(
-              'Track down transactions to connect the dots between 10-second page loads and poor-performing API calls or slow database queries.'
-            ),
-            link: 'https://docs.sentry.io/platforms/javascript/guides/bun/tracing/',
-          },
-        ],
+  nextSteps: () => [],
 };
 
 const customMetricsOnboarding: OnboardingConfig = {

+ 0 - 8
static/app/gettingStartedDocs/java/java.tsx

@@ -286,14 +286,6 @@ const onboarding: OnboardingConfig<PlatformOptions> = {
       description: t('Check out our sample applications.'),
       link: 'https://github.com/getsentry/sentry-java/tree/main/sentry-samples',
     },
-    {
-      id: 'performance-monitoring',
-      name: t('Tracing'),
-      description: t(
-        'Stay ahead of latency issues and trace every slow transaction to a poor-performing API call or database query.'
-      ),
-      link: 'https://docs.sentry.io/platforms/java/tracing/',
-    },
   ],
 };
 

+ 0 - 8
static/app/gettingStartedDocs/java/spring-boot.tsx

@@ -288,14 +288,6 @@ const onboarding: OnboardingConfig<PlatformOptions> = {
       description: t('Check out our sample applications.'),
       link: 'https://github.com/getsentry/sentry-java/tree/main/sentry-samples',
     },
-    {
-      id: 'performance-monitoring',
-      name: t('Tracing'),
-      description: t(
-        'Stay ahead of latency issues and trace every slow transaction to a poor-performing API call or database query.'
-      ),
-      link: 'https://docs.sentry.io/platforms/java/guides/spring-boot/tracing/',
-    },
   ],
 };
 

Некоторые файлы не были показаны из-за большого количества измененных файлов