Merge pull request uselagoon#3719 from uselagoon/refactor-group-membe…

…rcount

refactor: use existing cached member check to return membercount

services/api/src/models/group.ts

@@ -539,16 +539,8 @@ export const Group = (clients: {
   const getGroupMemberCount = async (
     group: Group
   ): Promise<number> => {
-    const roleSubgroups = group.subGroups.filter(isRoleSubgroup);
-    let membership = 0;
-    for (const roleSubgroup of roleSubgroups) {
-      const keycloakUsers = await keycloakAdminClient.groups.listMembers({
-        id: roleSubgroup.id
-      });
-
-      membership = membership + keycloakUsers.length;
-    }
-    return membership;
+    const membership = await getGroupMembership(group)
+    return membership.length;
   };
 
   const addGroup = async (groupInput: Group, projectId?: number, organizationId?: number): Promise<Group> => {