Skip generateKmqDataTables on migration disabled instances

Brainicism · Feb 2, 2024 · 4bf7163 · 4bf7163
1 parent b54a997
commit 4bf7163
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 12 deletions.
diff --git a/src/seed/bootstrap.ts b/src/seed/bootstrap.ts
@@ -207,7 +207,13 @@ async function bootstrapDatabases(): Promise<void> {
     }
 
     if (process.env.NODE_ENV === EnvType.PROD) {
-        await generateKmqDataTables(db);
+        if (!KmqConfiguration.Instance.disallowMigrations()) {
+            await generateKmqDataTables(db);
+        } else {
+            logger.info(
+                "Skipping generateKmqDataTables due to disabled migrations",
+            );
+        }
     }
 
     logger.info("Cleaning up stale data");

diff --git a/src/seed/seed_db.ts b/src/seed/seed_db.ts
@@ -99,6 +99,7 @@ async function replaceBetterAudioSongs(
             "b.vlink as better_audio_link",
         ])
         .where("b.vlink", "is not", null)
+        .$narrowType<{ better_audio_link: string }>()
         .execute();
 
     for (const betterAudioSong of songsWithBetterAudioCounterpart) {
@@ -109,17 +110,13 @@ async function replaceBetterAudioSongs(
             .execute();
 
         // replace main video with better audio entry
-        if (betterAudioSong.better_audio_link) {
-            // TODO: this null check shouldn't be needed, but Kysely does not narrow types automatically
-            // can be removed when .$narrowType is available
-            await db
-                .updateTable("app_kpop")
-                .where("id", "=", betterAudioSong.original_id)
-                .set({
-                    vlink: betterAudioSong.better_audio_link,
-                })
-                .execute();
-        }
+        await db
+            .updateTable("app_kpop")
+            .where("id", "=", betterAudioSong.original_id)
+            .set({
+                vlink: betterAudioSong.better_audio_link,
+            })
+            .execute();
     }
 }