From cd718488000e1c52309bac9d0d33e7657c387baf Mon Sep 17 00:00:00 2001
From: Carlos Monastyrski <charly.monastyrski@gmail.com>
Date: Fri, 25 Jul 2025 13:10:54 -0300
Subject: [PATCH] Avoid migrating existing reminders

---
 .../20250725144940_fix-secret-reminders-migration.ts          | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/backend/src/db/migrations/20250725144940_fix-secret-reminders-migration.ts b/backend/src/db/migrations/20250725144940_fix-secret-reminders-migration.ts
index 81e8ccf31..35612f2d7 100644
--- a/backend/src/db/migrations/20250725144940_fix-secret-reminders-migration.ts
+++ b/backend/src/db/migrations/20250725144940_fix-secret-reminders-migration.ts
@@ -28,6 +28,10 @@ export async function up(knex: Knex): Promise<void> {
           .whereRaw(`"v2"."reminderRepeatDays" > 0`)
           .groupBy("v2.secretId");
       })
+      // Add LEFT JOIN with Reminder table to check for existing reminders
+      .leftJoin(TableName.Reminder, `${TableName.Reminder}.secretId`, `${TableName.SecretV2}.id`)
+      // Only include secrets that don't already have reminders
+      .whereNull(`${TableName.Reminder}.secretId`)
       .select(
         knex.ref("id").withSchema(TableName.SecretV2).as("secretId"),
         knex.ref("reminderRepeatDays").withSchema(TableName.SecretV2).as("reminderRepeatDays"),