Merge pull request #2605 from akhilmhdh/feat/permission-phase-2

feat: added filter folder to remove read only in migration
Infisical · Oct 17, 2024 · 13c0b31 · 13c0b31
2 parents 96bad7b + 99e65f7
commit 13c0b31
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/backend/src/ee/services/permission/project-permission.ts b/backend/src/ee/services/permission/project-permission.ts
@@ -798,11 +798,13 @@ export const backfillPermissionV1SchemaToV2Schema = (data: z.infer<typeof Projec
     };
   });
 
-  return formattedData.concat(
-    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
-    // @ts-ignore-error this is valid ts
-    secretImportPolicies,
-    dynamicSecretPolicies,
-    hasReadOnlyFolder.length ? [] : secretFolderPolicies
-  );
+  return formattedData
+    .filter((i) => i.subject !== ProjectPermissionSub.SecretFolders)
+    .concat(
+      // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+      // @ts-ignore-error this is valid ts
+      secretImportPolicies,
+      dynamicSecretPolicies,
+      hasReadOnlyFolder.length ? [] : secretFolderPolicies
+    );
 };