🐝 (baking) report explorers with missing variable IDs (#3552)

owid · May 6, 2024 · 7d5fb2c · 7d5fb2c
1 parent 2e803e6
commit 7d5fb2c
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/baker/siteRenderers.tsx b/baker/siteRenderers.tsx
@@ -735,6 +735,18 @@ export const renderExplorerPage = async (
             `SELECT id, grapherConfigETL, grapherConfigAdmin FROM variables WHERE id IN (?)`,
             [requiredVariableIds]
         )
+
+        // check if all required variable IDs exist in the database
+        const missingIds = requiredVariableIds.filter(
+            (id) => !partialGrapherConfigRows.find((row) => row.id === id)
+        )
+        if (missingIds.length > 0) {
+            void logErrorAndMaybeSendToBugsnag(
+                new JsonError(
+                    `Referenced variable IDs do not exist in the database for explorer ${program.slug}: ${missingIds.join(", ")}.`
+                )
+            )
+        }
     }
 
     const parseGrapherConfigFromRow = (row: ChartRow): GrapherInterface => {