From 49944c223fb4883d1e7e78be072c3481ab4dd14f Mon Sep 17 00:00:00 2001
From: Alan <alannnc@gmail.com>
Date: Mon, 11 Sep 2023 00:10:42 -0700
Subject: [PATCH] Fix  types and temporarily hiding pagination

---
 apps/web/pages/event-types/index.tsx                | 13 ++++++++++---
 .../viewer/eventTypes/infiniteEventTypes.handler.ts |  7 ++++---
 .../server/routers/viewer/teams/list.handler.ts     | 12 ++++++++++--
 3 files changed, 24 insertions(+), 8 deletions(-)

diff --git a/apps/web/pages/event-types/index.tsx b/apps/web/pages/event-types/index.tsx
index 738c642e72..fbb4f962cb 100644
--- a/apps/web/pages/event-types/index.tsx
+++ b/apps/web/pages/event-types/index.tsx
@@ -896,6 +896,9 @@ const Main = ({ filters }: { filters: ReturnType<typeof getTeamsFiltersFromQuery
   const { data: teams } = trpc.viewer.teams.list.useQuery(undefined, {
     // Teams don't change that frequently
     refetchOnWindowFocus: false,
+    trpc: {
+      context: { skipBatch: true },
+    },
   });
 
   // After teams are fetched we then load event types for each team
@@ -952,8 +955,10 @@ const Main = ({ filters }: { filters: ReturnType<typeof getTeamsFiltersFromQuery
               if (!teamEventTypes || teamEventTypes.length === 0 || !firstElementTeamEventTypes.team) {
                 return null;
               }
-              const teamDataMatchWithEventType = teams && teams.length > 0 && teams[index];
-              const membershipCount = teamDataMatchWithEventType?.membershipCount || 0;
+              const teamDataMatchWithEventType = teams && teams.length > 0 ? teams[index] : null;
+              const membershipCount = teamDataMatchWithEventType
+                ? teamDataMatchWithEventType?.membershipCount
+                : 0;
 
               return (
                 <>
@@ -970,7 +975,9 @@ const Main = ({ filters }: { filters: ReturnType<typeof getTeamsFiltersFromQuery
                     <EventTypeList
                       data={teamEventTypes}
                       key={index}
-                      readonly={"MEMBER" === teamDataMatchWithEventType.role}
+                      readonly={
+                        teamDataMatchWithEventType ? "MEMBER" === teamDataMatchWithEventType.role : false
+                      }
                     />
                   ) : (
                     <EmptyEventTypeList
diff --git a/packages/trpc/server/routers/viewer/eventTypes/infiniteEventTypes.handler.ts b/packages/trpc/server/routers/viewer/eventTypes/infiniteEventTypes.handler.ts
index 860b369279..a96a10d607 100644
--- a/packages/trpc/server/routers/viewer/eventTypes/infiniteEventTypes.handler.ts
+++ b/packages/trpc/server/routers/viewer/eventTypes/infiniteEventTypes.handler.ts
@@ -38,7 +38,7 @@ export const paginateHandler = async ({ ctx, input }: EventTypesPaginateProps) =
     teamConditional = { teamId: { in: teamIds } };
   }
 
-  const skip = (page - 1) * pageSize;
+  // const skip = (page - 1) * pageSize;
 
   const result = await prisma.eventType.findMany({
     where: {
@@ -83,8 +83,9 @@ export const paginateHandler = async ({ ctx, input }: EventTypesPaginateProps) =
         },
       },
     },
-    skip,
-    take: pageSize,
+    // Temporarily disabled until we can figure out how to do this properly
+    // skip,
+    // take: pageSize,
   });
 
   return result;
diff --git a/packages/trpc/server/routers/viewer/teams/list.handler.ts b/packages/trpc/server/routers/viewer/teams/list.handler.ts
index 3046f05ac2..a7f92b34f5 100644
--- a/packages/trpc/server/routers/viewer/teams/list.handler.ts
+++ b/packages/trpc/server/routers/viewer/teams/list.handler.ts
@@ -35,6 +35,15 @@ export const listHandler = async ({ ctx }: ListOptions) => {
 
     const isOrgAdmin = !!(await isOrganisationAdmin(ctx.user.id, ctx.user.organization.id)); // Org id exists here as we're inside a conditional TS complaining for some reason
 
+    // This can be optimized by using a custom view between membership and team and teamMemberCount
+    const membershipCount = await prisma.teamMemberCount.findMany({
+      where: {
+        id: {
+          in: membershipsWithoutParent.map((m) => m.teamId),
+        },
+      },
+    });
+
     return membershipsWithoutParent.map(({ team: { inviteTokens, ..._team }, ...membership }) => ({
       role: membership.role,
       accepted: membership.accepted,
@@ -42,6 +51,7 @@ export const listHandler = async ({ ctx }: ListOptions) => {
       ..._team,
       /** To prevent breaking we only return non-email attached token here, if we have one */
       inviteToken: inviteTokens.find((token) => token.identifier === "invite-link-for-teamId-" + _team.id),
+      membershipCount: membershipCount.find((m) => m.id === _team.id)?.count || 0,
     }));
   }
 
@@ -68,8 +78,6 @@ export const listHandler = async ({ ctx }: ListOptions) => {
     },
   });
 
-  console.log({ membershipCount });
-
   return memberships
     .filter((mmship) => {
       const metadata = teamMetadataSchema.parse(mmship.team.metadata);