From b1bb8a6539f123556231f1cbc5cee6047aee33e5 Mon Sep 17 00:00:00 2001
From: Henry <henry_ball@hotmail.co.uk>
Date: Tue, 27 May 2025 22:09:12 +0100
Subject: [PATCH] fix: create card.updated.list activity when changing list

---
 packages/api/src/routers/card.ts        | 14 ++++++++++++--
 packages/db/src/repository/card.repo.ts |  1 +
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/packages/api/src/routers/card.ts b/packages/api/src/routers/card.ts
index 7cfdc2fa..d3d9da3a 100644
--- a/packages/api/src/routers/card.ts
+++ b/packages/api/src/routers/card.ts
@@ -720,7 +720,7 @@ export const cardRouter = createTRPCRouter({
 
       const activities = [];
 
-      if (existingCard.title !== input.title) {
+      if (input.title && existingCard.title !== input.title) {
         activities.push({
           type: "card.updated.title" as const,
           cardId: result.id,
@@ -730,7 +730,7 @@ export const cardRouter = createTRPCRouter({
         });
       }
 
-      if (existingCard.description !== input.description) {
+      if (input.description && existingCard.description !== input.description) {
         activities.push({
           type: "card.updated.description" as const,
           cardId: result.id,
@@ -740,6 +740,16 @@ export const cardRouter = createTRPCRouter({
         });
       }
 
+      if (newListId && existingCard.listId !== newListId) {
+        activities.push({
+          type: "card.updated.list" as const,
+          cardId: result.id,
+          createdBy: userId,
+          fromListId: existingCard.listId,
+          toListId: newListId,
+        });
+      }
+
       if (activities.length > 0) {
         await cardActivityRepo.bulkCreate(ctx.db, activities);
       }
diff --git a/packages/db/src/repository/card.repo.ts b/packages/db/src/repository/card.repo.ts
index c36a9fe3..60bbbf6c 100644
--- a/packages/db/src/repository/card.repo.ts
+++ b/packages/db/src/repository/card.repo.ts
@@ -187,6 +187,7 @@ export const getByPublicId = (db: dbClient, cardPublicId: string) => {
       publicId: true,
       title: true,
       description: true,
+      listId: true,
     },
     where: eq(cards.publicId, cardPublicId),
   });