chore: clean client api flag removed (#4368)

Unleash · Jul 31, 2023 · ce468dc · ce468dc
1 parent b34f00c
commit ce468dc
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 35 deletions.
diff --git a/src/lib/__snapshots__/create-config.test.ts.snap b/src/lib/__snapshots__/create-config.test.ts.snap
@@ -69,7 +69,6 @@ exports[`should create default config 1`] = `
     "flags": {
       "anonymiseEventLog": false,
       "caseInsensitiveInOperators": false,
-      "cleanClientApi": false,
       "configurableFeatureTypeLifetimes": false,
       "customRootRoles": false,
       "demo": false,
@@ -104,7 +103,6 @@ exports[`should create default config 1`] = `
     "experiments": {
       "anonymiseEventLog": false,
       "caseInsensitiveInOperators": false,
-      "cleanClientApi": false,
       "configurableFeatureTypeLifetimes": false,
       "customRootRoles": false,
       "demo": false,

diff --git a/src/lib/services/feature-toggle-service.ts b/src/lib/services/feature-toggle-service.ts
@@ -940,37 +940,29 @@ class FeatureToggleService {
         const result = await this.featureToggleClientStore.getClient(
             query || {},
         );
-        if (this.flagResolver.isEnabled('cleanClientApi')) {
-            return result.map(
-                ({
-                    name,
-                    type,
-                    enabled,
-                    project,
-                    stale,
-                    strategies,
-                    variants,
-                    description,
-                    createdAt,
-                    lastSeenAt,
-                    impressionData,
-                }) => ({
-                    name,
-                    type,
-                    enabled,
-                    project,
-                    stale,
-                    strategies,
-                    variants,
-                    description,
-                    createdAt,
-                    lastSeenAt,
-                    impressionData,
-                }),
-            );
-        } else {
-            return result;
-        }
+        return result.map(
+            ({
+                name,
+                type,
+                enabled,
+                project,
+                stale,
+                strategies,
+                variants,
+                description,
+                impressionData,
+            }) => ({
+                name,
+                type,
+                enabled,
+                project,
+                stale,
+                strategies,
+                variants,
+                description,
+                impressionData,
+            }),
+        );
     }
 
     async getPlaygroundFeatures(

diff --git a/src/lib/types/experimental.ts b/src/lib/types/experimental.ts
@@ -14,7 +14,6 @@ export type IFlagKey =
     | 'strictSchemaValidation'
     | 'proPlanAutoCharge'
     | 'personalAccessTokensKillSwitch'
-    | 'cleanClientApi'
     | 'migrationLock'
     | 'demo'
     | 'googleAuthEnabled'
@@ -81,7 +80,6 @@ const flags: IFlags = {
         process.env.UNLEASH_PAT_KILL_SWITCH,
         false,
     ),
-    cleanClientApi: parseEnvVarBoolean(process.env.CLEAN_CLIENT_API, false),
     migrationLock: parseEnvVarBoolean(process.env.MIGRATION_LOCK, false),
     demo: parseEnvVarBoolean(process.env.UNLEASH_DEMO, false),
     googleAuthEnabled: parseEnvVarBoolean(