bugfix(hasura-auth) keep select and delete permissions already set in…

… metadata for auth tables instead of overriding them
nhost · Jul 12, 2023 · 9d40d77 · 9d40d77
1 parent 87fee13
commit 9d40d77
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/utils/hasura-metadata/helpers.ts b/src/utils/hasura-metadata/helpers.ts
@@ -115,11 +115,17 @@ export const patchTableObject = (
   }
 
   if (select_permissions) {
-    existingTable.select_permissions = [...select_permissions];
+    const mergedSelectPermissions = [...(existingTable.select_permissions ?? []), ...select_permissions];
+    existingTable.select_permissions = [
+      ...new Map(mergedSelectPermissions.map((permission) => [permission.role, permission])).values()
+    ];
   }
 
   if (delete_permissions) {
-    existingTable.delete_permissions = [...delete_permissions];
+    const mergedDeletePermissions = [...(existingTable.delete_permissions ?? []), ...delete_permissions];
+    existingTable.delete_permissions = [
+      ...new Map(mergedDeletePermissions.map((permission) => [permission.role, permission])).values()
+    ];
   }
 
   // TODO merge other fields (permissions, computed fields, etc.) - not required by Hasura-auth yet