Browse Source

chore(dashboard): Alpha badge the releases dataset (#35208)

Add alpha badge to the dataset selector
Shruthi 2 years ago
parent
commit
e7228939a5

+ 8 - 1
static/app/views/dashboardsV2/widgetBuilder/buildSteps/dataSetStep.tsx

@@ -1,5 +1,7 @@
+import {Fragment} from 'react';
 import styled from '@emotion/styled';
 
+import FeatureBadge from 'sentry/components/featureBadge';
 import RadioGroup, {RadioGroupProps} from 'sentry/components/forms/controls/radioGroup';
 import ExternalLink from 'sentry/components/links/externalLink';
 import {t, tct} from 'sentry/locale';
@@ -65,7 +67,12 @@ export function DataSetStep({
           hasReleaseHealthFeature
             ? [
                 ...DATASET_CHOICES,
-                [DataSet.RELEASES, t('Releases (sessions, crash rates)')],
+                [
+                  DataSet.RELEASES,
+                  <Fragment key="releases-dataset">
+                    {t('Releases (sessions, crash rates)')} <FeatureBadge type="alpha" />
+                  </Fragment>,
+                ],
               ]
             : DATASET_CHOICES
         }

+ 2 - 8
tests/js/spec/views/dashboardsV2/widgetBuilder/widgetBuilder.spec.tsx

@@ -2616,7 +2616,7 @@ describe('WidgetBuilder', function () {
         userEvent.click(await screen.findByText('Releases (sessions, crash rates)'));
 
         expect(metricsDataMock).toHaveBeenCalled();
-        expect(screen.getByLabelText('Releases (sessions, crash rates)')).toBeChecked();
+        expect(screen.getByLabelText(/Releases/i)).toBeChecked();
       });
 
       it('does not display "add an equation" button', async function () {
@@ -2674,13 +2674,7 @@ describe('WidgetBuilder', function () {
         userEvent.click(await screen.findByText('Table'));
         userEvent.click(screen.getByText('World Map'));
 
-        await waitFor(() =>
-          expect(
-            screen.getByRole('radio', {
-              name: 'Select Releases (sessions, crash rates)',
-            })
-          ).toBeDisabled()
-        );
+        await waitFor(() => expect(screen.getByLabelText(/Releases/)).toBeDisabled());
 
         expect(
           screen.getByRole('radio', {