fix: allow removing item from queue if no labels

lmnr-ai · Jan 6, 2025 · b5d08f7 · b5d08f7
1 parent 4db0162
commit b5d08f7
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/frontend/app/api/projects/[projectId]/queues/[queueId]/remove/route.ts b/frontend/app/api/projects/[projectId]/queues/[queueId]/remove/route.ts
@@ -30,7 +30,7 @@ const removeQueueItemSchema = z.object({
       id: z.string()
     }),
     reasoning: z.string().optional().nullable()
-  })).nonempty(),
+  })),
   action: z.null().or(z.object({
     resultId: z.string().optional(),
     datasetId: z.string().optional()
@@ -61,14 +61,16 @@ export async function POST(request: Request, { params }: { params: { projectId:
     labelSource: "MANUAL" as const,
   }));
 
-  const insertedLabels = await db.insert(labels).values(newLabels).onConflictDoUpdate({
-    target: [labels.spanId, labels.classId, labels.userId],
-    set: {
-      value: sql`excluded.value`,
-      labelSource: sql`excluded.label_source`,
-      reasoning: sql`COALESCE(excluded.reasoning, labels.reasoning)`,
-    }
-  }).returning();
+  const insertedLabels = newLabels.length > 0
+    ? await db.insert(labels).values(newLabels).onConflictDoUpdate({
+      target: [labels.spanId, labels.classId, labels.userId],
+      set: {
+        value: sql`excluded.value`,
+        labelSource: sql`excluded.label_source`,
+        reasoning: sql`COALESCE(excluded.reasoning, labels.reasoning)`,
+      }
+    }).returning()
+    : [];
 
   if (action?.resultId) {
     const resultId = action.resultId;

diff --git a/frontend/app/api/projects/[projectId]/queues/route.ts b/frontend/app/api/projects/[projectId]/queues/route.ts
@@ -11,8 +11,6 @@ export async function POST(
 ): Promise<Response> {
   const projectId = params.projectId;
 
-
-
   const body = await req.json();
   const { name } = body;