Browse Source

chore(dashboards): Remove releases on widget charts ff (#81317)

<!-- Describe your PR here. -->
removing all instances of the `dashboards-releases-on-charts` ff because
it has been fully rolled out.
Nikki Kapadia 3 months ago
parent
commit
d66e44f3a8

+ 5 - 0
static/app/components/modals/widgetBuilder/addToDashboardModal.spec.tsx

@@ -97,6 +97,11 @@ describe('add to dashboard modal', () => {
       body: testDashboard,
     });
 
+    MockApiClient.addMockResponse({
+      url: '/organizations/org-slug/releases/stats/',
+      body: [],
+    });
+
     eventsStatsMock = MockApiClient.addMockResponse({
       url: '/organizations/org-slug/events-stats/',
       body: [],

+ 0 - 1
static/app/components/modals/widgetBuilder/addToDashboardModal.tsx

@@ -306,7 +306,6 @@ function AddToDashboardModal({
                     widgetLegendState={widgetLegendState}
                     onLegendSelectChanged={() => {}}
                     legendOptions={
-                      organization.features.includes('dashboards-releases-on-charts') &&
                       widgetLegendState.widgetRequiresLegendUnselection(widget)
                         ? {selected: unselectedReleasesForCharts}
                         : undefined

+ 5 - 0
static/app/components/modals/widgetViewerModal.spec.tsx

@@ -131,6 +131,11 @@ describe('Modals -> WidgetViewerModal', function () {
       body: [],
     });
 
+    MockApiClient.addMockResponse({
+      url: '/organizations/org-slug/releases/stats/',
+      body: [],
+    });
+
     eventsMetaMock = MockApiClient.addMockResponse({
       url: '/organizations/org-slug/events-meta/',
       body: {count: 33323612},

+ 4 - 0
static/app/views/dashboards/dashboard.spec.tsx

@@ -86,6 +86,10 @@ describe('Dashboards > Dashboard', () => {
       method: 'POST',
       body: [],
     });
+    MockApiClient.addMockResponse({
+      url: `/organizations/org-slug/releases/stats/`,
+      body: [],
+    });
     MockApiClient.addMockResponse({
       url: '/organizations/org-slug/events-stats/',
       method: 'GET',

+ 8 - 0
static/app/views/dashboards/detail.spec.tsx

@@ -100,6 +100,10 @@ describe('Dashboards > Detail', function () {
         url: '/organizations/org-slug/releases/',
         body: [],
       });
+      MockApiClient.addMockResponse({
+        url: '/organizations/org-slug/releases/stats/',
+        body: [],
+      });
       MockApiClient.addMockResponse({
         url: '/organizations/org-slug/metrics/meta/',
         body: [],
@@ -368,6 +372,10 @@ describe('Dashboards > Detail', function () {
         url: '/organizations/org-slug/releases/',
         body: [],
       });
+      MockApiClient.addMockResponse({
+        url: '/organizations/org-slug/releases/stats/',
+        body: [],
+      });
       MockApiClient.addMockResponse({
         url: '/organizations/org-slug/sdk-updates/',
         body: [],

+ 0 - 1
static/app/views/dashboards/widgetBuilder/buildSteps/visualizationStep.tsx

@@ -141,7 +141,6 @@ export function VisualizationStep({
           shouldResize={false}
           onLegendSelectChanged={() => {}}
           legendOptions={
-            organization.features.includes('dashboards-releases-on-charts') &&
             widgetLegendState.widgetRequiresLegendUnselection(widget)
               ? {selected: unselectedReleasesForCharts}
               : undefined

+ 4 - 0
static/app/views/dashboards/widgetBuilder/widgetBuilder.spec.tsx

@@ -265,6 +265,10 @@ describe('WidgetBuilder', function () {
       url: '/organizations/org-slug/releases/',
       body: [],
     });
+    MockApiClient.addMockResponse({
+      url: '/organizations/org-slug/releases/stats/',
+      body: [],
+    });
     MockApiClient.addMockResponse({
       url: `/organizations/org-slug/spans/fields/`,
       body: [],

+ 4 - 0
static/app/views/dashboards/widgetBuilder/widgetBuilderDataset.spec.tsx

@@ -276,6 +276,10 @@ describe('WidgetBuilder', function () {
       url: '/organizations/org-slug/releases/',
       body: [],
     });
+    MockApiClient.addMockResponse({
+      url: '/organizations/org-slug/releases/stats/',
+      body: [],
+    });
     MockApiClient.addMockResponse({
       url: `/organizations/org-slug/spans/fields/`,
       body: [],

+ 4 - 0
static/app/views/dashboards/widgetBuilder/widgetBuilderSortBy.spec.tsx

@@ -236,6 +236,10 @@ describe('WidgetBuilder', function () {
       url: '/organizations/org-slug/releases/',
       body: [],
     });
+    MockApiClient.addMockResponse({
+      url: '/organizations/org-slug/releases/stats/',
+      body: [],
+    });
     MockApiClient.addMockResponse({
       url: `/organizations/org-slug/spans/fields/`,
       body: [],

+ 1 - 3
static/app/views/dashboards/widgetCard/chart.tsx

@@ -297,7 +297,6 @@ class WidgetCardChart extends Component<WidgetCardChartProps> {
       chartZoomOptions,
       timeseriesResultsTypes,
       shouldResize,
-      organization,
     } = this.props;
 
     if (widget.displayType === 'table') {
@@ -524,8 +523,7 @@ class WidgetCardChart extends Component<WidgetCardChartProps> {
 
           const forwardedRef = this.props.chartGroup ? this.handleRef : undefined;
 
-          return organization.features.includes('dashboards-releases-on-charts') &&
-            widgetLegendState.widgetRequiresLegendUnselection(widget) ? (
+          return widgetLegendState.widgetRequiresLegendUnselection(widget) ? (
             <ReleaseSeries
               end={end}
               start={start}

Some files were not shown because too many files changed in this diff