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

Experiment with simplifications (#75231)

Want feedback from the team, extra for example I though wasn't necessary

![Screenshot 2024-07-30 at 16 38
29](https://github.com/user-attachments/assets/582d33e2-4fc9-4714-a7f2-acc8d6889a88)
//the dsn is useless ignore

---------

Co-authored-by: Ivana Kellyer <ivana.kellyerova@sentry.io>
Co-authored-by: Priscila Oliveira <priscila.oliveira@sentry.io>
Steven Eubank 7 месяцев назад
Родитель
Сommit
0219889485

+ 1 - 3
static/app/components/updatedEmptyState.spec.tsx

@@ -31,9 +31,7 @@ describe('UpdatedEmptyState', function () {
     expect(await screen.findByText('Preview a Sentry Issue')).toBeInTheDocument();
 
     expect(
-      await screen.findByText(
-        textWithMarkupMatcher('Install sentry-sdk from PyPI with the django extra')
-      )
+      await screen.findByText(textWithMarkupMatcher('Install sentry-sdk from PyPI:'))
     ).toBeInTheDocument();
 
     await userEvent.click(screen.getByRole('button', {name: 'Next'}));

+ 1 - 3
static/app/gettingStartedDocs/python/django.spec.tsx

@@ -15,9 +15,7 @@ describe('django onboarding docs', function () {
 
     // Renders install instructions
     expect(
-      screen.getByText(
-        textWithMarkupMatcher(/pip install --upgrade 'sentry-sdk\[django\]'/)
-      )
+      screen.getByText(textWithMarkupMatcher('pip install --upgrade sentry-sdk'))
     ).toBeInTheDocument();
   });
 

+ 10 - 27
static/app/gettingStartedDocs/python/django.tsx

@@ -12,7 +12,7 @@ import {t, tct} from 'sentry/locale';
 
 type Params = DocsParams;
 
-const getInstallSnippet = () => `pip install --upgrade 'sentry-sdk[django]'`;
+const getInstallSnippet = () => `pip install --upgrade sentry-sdk`;
 
 const getSdkSetupSnippet = (params: Params) => `
 # settings.py
@@ -39,31 +39,14 @@ sentry_sdk.init(
 `;
 
 const onboarding: OnboardingConfig = {
-  introduction: () =>
-    tct('The Django integration adds support for the [link:Django Web Framework].', {
-      link: <ExternalLink href="https://www.djangoproject.com/" />,
-    }),
-  install: (params: Params) => [
+  install: () => [
     {
       type: StepType.INSTALL,
-      description: tct(
-        'Install [code:sentry-sdk] from PyPI with the [sentryDjangoCode:django] extra:',
-        {
-          code: <code />,
-          sentryDjangoCode: <code />,
-        }
-      ),
+      description: tct('Install [code:sentry-sdk] from PyPI:', {
+        code: <code />,
+      }),
       configurations: [
         {
-          description: params.isProfilingSelected
-            ? tct(
-                'You need a minimum version [codeVersion:1.18.0] of the [codePackage:sentry-python] SDK for the profiling feature.',
-                {
-                  codeVersion: <code />,
-                  codePackage: <code />,
-                }
-              )
-            : undefined,
           language: 'bash',
           code: getInstallSnippet(),
         },
@@ -74,7 +57,7 @@ const onboarding: OnboardingConfig = {
     {
       type: StepType.CONFIGURE,
       description: tct(
-        'If you have the [codeDjango:django] package in your dependencies, the Django integration will be enabled automatically when you initialize the Sentry SDK. Initialize the Sentry SDK in your Django [codeSettings:settings.py] file:',
+        'Initialize the Sentry SDK in your Django [codeSettings:settings.py] file:',
         {
           codeDjango: <code />,
           codeSettings: <code />,
@@ -115,18 +98,18 @@ urlpatterns = [
         <div>
           <p>
             {tct(
-              'When you point your browser to [link:http://localhost:8000/sentry-debug/] a transaction in the Performance section of Sentry will be created.',
+              'When you point your browser to [link:http://localhost:8000/sentry-debug/] an error with a trace will be created. So you can explore errors and tracing portions of Sentry.',
               {
-                link: <ExternalLink href="http://localhost:8000/" />,
+                link: <ExternalLink href="http://localhost:8000/sentry-debug/" />,
               }
             )}
           </p>
+          <br />
           <p>
             {t(
-              'Additionally, an error event will be sent to Sentry and will be connected to the transaction.'
+              'It can take a couple of moments for the data to appear in Sentry. Bear with us, the internet is huge.'
             )}
           </p>
-          <p>{t('It takes a couple of moments for the data to appear in Sentry.')}</p>
         </div>
       ),
     },