19306: Increase sharing dialog query limit and dedupe uuids when fetching user details 19306-share-dialog-limit
authorStephen Smith <stephen@curii.com>
Fri, 14 Oct 2022 19:49:17 +0000 (15:49 -0400)
committerStephen Smith <stephen@curii.com>
Fri, 14 Oct 2022 19:49:17 +0000 (15:49 -0400)
Arvados-DCO-1.1-Signed-off-by: Stephen Smith <stephen@curii.com>

src/store/sharing-dialog/sharing-dialog-actions.ts

index cdc6c0c7267d4bab58434999b260d8905de7bc82..c0fdeda5a74a546ee7c8c350f72f3d9b3c413dba 100644 (file)
@@ -152,11 +152,11 @@ export const initializeManagementForm = async (dispatch: Dispatch, getState: ()
         const { items: permissionLinks } = await permissionService.listResourcePermissions(resourceUuid);
         dispatch<any>(initializePublicAccessForm(permissionLinks));
         const filters = new FilterBuilder()
-            .addIn('uuid', permissionLinks.map(({ tailUuid }) => tailUuid))
+            .addIn('uuid', Array.from(new Set(permissionLinks.map(({ tailUuid }) => tailUuid))))
             .getFilters();
 
-        const { items: users } = await userService.list({ filters, count: "none" });
-        const { items: groups } = await groupsService.list({ filters, count: "none" });
+        const { items: users } = await userService.list({ filters, count: "none", limit: 1000 });
+        const { items: groups } = await groupsService.list({ filters, count: "none", limit: 1000 });
 
         const getEmail = (tailUuid: string) => {
             const user = users.find(({ uuid }) => uuid === tailUuid);