index on the admin table

webdevcody · Oct 18, 2024 · 25cb787 · 25cb787
1 parent 22cd75c
commit 25cb787
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 10 deletions.
diff --git a/app/play/page.tsx b/app/play/page.tsx
@@ -106,7 +106,6 @@ export default function PlayPage() {
 
       <div className="grid grid-cols-1 md:grid-cols-2 lg:grid-cols-3 gap-8">
         {filteredMaps.map((map) => (
-          <Card key={map._id} className="flex flex-col h-full">
           <Card
             key={map._id}
             className={cn(

diff --git a/convex/maps.ts b/convex/maps.ts
@@ -218,10 +218,12 @@ export const getMaps = query({
         return null;
       }
 
-      const admins = await ctx.db.query("admins").collect();
-      const isAdmin = admins.some((admin) => admin.userId === userId);
+      const admin = await ctx.db
+        .query("admins")
+        .withIndex("by_userId", (q) => q.eq("userId", userId))
+        .first();
 
-      if (isAdmin) {
+      if (admin) {
         return await ctx.db
           .query("maps")
           .filter((q) => q.eq(q.field("isReviewed"), args.isReviewed))
@@ -250,10 +252,12 @@ export const approveMap = mutation({
       throw new Error("Unauthorized");
     }
 
-    const admins = await ctx.db.query("admins").collect();
-    const isAdmin = admins.some((admin) => admin.userId === userId);
+    const admin = await ctx.db
+      .query("admins")
+      .withIndex("by_userId", (q) => q.eq("userId", userId))
+      .first();
 
-    if (!isAdmin) {
+    if (!admin) {
       throw new Error("Unauthorized");
     } else {
       const maps = await ctx.db.query("maps").collect();

diff --git a/convex/schema.ts b/convex/schema.ts
@@ -64,5 +64,5 @@ export default defineSchema({
   }).index("by_mapId_userId", ["mapId", "userId"]),
   admins: defineTable({
     userId: v.id("users"),
-  }),
+  }).index("by_userId", ["userId"]),
 });
diff --git a/convex/users.ts b/convex/users.ts
@@ -34,8 +34,11 @@ export const isAdmin = query({
       return false;
     }
 
-    const admins = await ctx.db.query("admins").collect();
+    const admin = await ctx.db
+      .query("admins")
+      .withIndex("by_userId", (q) => q.eq("userId", userId))
+      .first();
 
-    return admins.some((admin) => admin.userId === userId);
+    return !!admin;
   },
 });