From e34c9bc0bf9ef65dd48a5fe2e8054bb04109b891 Mon Sep 17 00:00:00 2001
From: Mateusz Kwasniewski <kwasniewski.mateusz@gmail.com>
Date: Mon, 22 May 2023 13:31:31 +0200
Subject: [PATCH] feat: disable bulk toggles flag (#3827)

---
 .../ProjectFeaturesBatchActions.tsx           | 43 +++++++++++++------
 frontend/src/interfaces/uiConfig.ts           |  1 +
 .../admin-api/project/project-features.ts     | 14 +++++-
 src/lib/types/experimental.ts                 |  3 +-
 .../__snapshots__/openapi.e2e.test.ts.snap    |  4 +-
 5 files changed, 46 insertions(+), 19 deletions(-)

diff --git a/frontend/src/component/project/Project/ProjectFeatureToggles/ProjectFeaturesBatchActions/ProjectFeaturesBatchActions.tsx b/frontend/src/component/project/Project/ProjectFeatureToggles/ProjectFeaturesBatchActions/ProjectFeaturesBatchActions.tsx
index f752fb7096..f8386e18dd 100644
--- a/frontend/src/component/project/Project/ProjectFeatureToggles/ProjectFeaturesBatchActions/ProjectFeaturesBatchActions.tsx
+++ b/frontend/src/component/project/Project/ProjectFeatureToggles/ProjectFeaturesBatchActions/ProjectFeaturesBatchActions.tsx
@@ -8,6 +8,8 @@ import { ManageTags } from './ManageTags';
 import { usePlausibleTracker } from 'hooks/usePlausibleTracker';
 import { BulkDisableDialog } from 'component/feature/FeatureToggleList/BulkDisableDialog';
 import { BulkEnableDialog } from 'component/feature/FeatureToggleList/BulkEnableDialog';
+import useUiConfig from 'hooks/api/getters/useUiConfig/useUiConfig';
+import { ConditionallyRender } from 'component/common/ConditionallyRender/ConditionallyRender';
 
 interface IProjectFeaturesBatchActionsProps {
     selectedIds: string[];
@@ -18,6 +20,7 @@ interface IProjectFeaturesBatchActionsProps {
 export const ProjectFeaturesBatchActions: FC<
     IProjectFeaturesBatchActionsProps
 > = ({ selectedIds, data, projectId }) => {
+    const { uiConfig } = useUiConfig();
     const [showExportDialog, setShowExportDialog] = useState(false);
     const [showBulkEnableDialog, setShowBulkEnableDialog] = useState(false);
     const [showBulkDisableDialog, setShowBulkDisableDialog] = useState(false);
@@ -59,20 +62,32 @@ export const ProjectFeaturesBatchActions: FC<
 
     return (
         <>
-            <Button
-                variant="outlined"
-                size="small"
-                onClick={() => setShowBulkEnableDialog(true)}
-            >
-                Enable
-            </Button>
-            <Button
-                variant="outlined"
-                size="small"
-                onClick={() => setShowBulkDisableDialog(true)}
-            >
-                Disable
-            </Button>
+            <ConditionallyRender
+                condition={Boolean(uiConfig?.flags?.disableBulkToggle)}
+                show={null}
+                elseShow={
+                    <Button
+                        variant="outlined"
+                        size="small"
+                        onClick={() => setShowBulkEnableDialog(true)}
+                    >
+                        Enable
+                    </Button>
+                }
+            />
+            <ConditionallyRender
+                condition={Boolean(uiConfig?.flags?.disableBulkToggle)}
+                show={null}
+                elseShow={
+                    <Button
+                        variant="outlined"
+                        size="small"
+                        onClick={() => setShowBulkDisableDialog(true)}
+                    >
+                        Disable
+                    </Button>
+                }
+            />
             <ArchiveButton projectId={projectId} features={selectedIds} />
             <Button
                 variant="outlined"
diff --git a/frontend/src/interfaces/uiConfig.ts b/frontend/src/interfaces/uiConfig.ts
index 7ad2fef0b8..36d2cbd464 100644
--- a/frontend/src/interfaces/uiConfig.ts
+++ b/frontend/src/interfaces/uiConfig.ts
@@ -50,6 +50,7 @@ export interface IFlags {
     googleAuthEnabled?: boolean;
     variantMetrics?: boolean;
     strategyImprovements?: boolean;
+    disableBulkToggle?: boolean;
 }
 
 export interface IVersionInfo {
diff --git a/src/lib/routes/admin-api/project/project-features.ts b/src/lib/routes/admin-api/project/project-features.ts
index ef4e4c005d..f1abee2acd 100644
--- a/src/lib/routes/admin-api/project/project-features.ts
+++ b/src/lib/routes/admin-api/project/project-features.ts
@@ -186,7 +186,7 @@ export default class ProjectFeaturesController extends Controller {
             permission: UPDATE_FEATURE_ENVIRONMENT,
             middleware: [
                 openApiService.validPath({
-                    tags: ['Unstable'],
+                    tags: ['Features'],
                     description:
                         'This endpoint enables multiple feature toggles.',
                     summary: 'Bulk enable a list of features.',
@@ -206,7 +206,7 @@ export default class ProjectFeaturesController extends Controller {
             permission: UPDATE_FEATURE_ENVIRONMENT,
             middleware: [
                 openApiService.validPath({
-                    tags: ['Unstable'],
+                    tags: ['Features'],
                     description:
                         'This endpoint disables multiple feature toggles.',
                     summary: 'Bulk disabled a list of features.',
@@ -748,6 +748,11 @@ export default class ProjectFeaturesController extends Controller {
         const { shouldActivateDisabledStrategies } = req.query;
         const { features } = req.body;
 
+        if (this.flagResolver.isEnabled('disableBulkToggle')) {
+            res.status(409).end();
+            return;
+        }
+
         await this.startTransaction(async (tx) =>
             this.transactionalFeatureToggleService(tx).bulkUpdateEnabled(
                 projectId,
@@ -775,6 +780,11 @@ export default class ProjectFeaturesController extends Controller {
         const { shouldActivateDisabledStrategies } = req.query;
         const { features } = req.body;
 
+        if (this.flagResolver.isEnabled('disableBulkToggle')) {
+            res.status(409).end();
+            return;
+        }
+
         await this.startTransaction(async (tx) =>
             this.transactionalFeatureToggleService(tx).bulkUpdateEnabled(
                 projectId,
diff --git a/src/lib/types/experimental.ts b/src/lib/types/experimental.ts
index 5464a8da60..6d0b157f4c 100644
--- a/src/lib/types/experimental.ts
+++ b/src/lib/types/experimental.ts
@@ -20,7 +20,8 @@ export type IFlagKey =
     | 'demo'
     | 'strategyImprovements'
     | 'googleAuthEnabled'
-    | 'variantMetrics';
+    | 'variantMetrics'
+    | 'disableBulkToggle';
 
 export type IFlags = Partial<{ [key in IFlagKey]: boolean | Variant }>;
 
diff --git a/src/test/e2e/api/openapi/__snapshots__/openapi.e2e.test.ts.snap b/src/test/e2e/api/openapi/__snapshots__/openapi.e2e.test.ts.snap
index bc7aeb9044..081499c35a 100644
--- a/src/test/e2e/api/openapi/__snapshots__/openapi.e2e.test.ts.snap
+++ b/src/test/e2e/api/openapi/__snapshots__/openapi.e2e.test.ts.snap
@@ -9678,7 +9678,7 @@ If the provided project does not exist, the list of events will be empty.",
         },
         "summary": "Bulk disabled a list of features.",
         "tags": [
-          "Unstable",
+          "Features",
         ],
       },
     },
@@ -9722,7 +9722,7 @@ If the provided project does not exist, the list of events will be empty.",
         },
         "summary": "Bulk enable a list of features.",
         "tags": [
-          "Unstable",
+          "Features",
         ],
       },
     },