Fix the actual bug

brimdata · Nov 13, 2024 · df44d87 · df44d87
1 parent 8f50cd1
commit df44d87
Showing 1 changed file with 15 additions and 10 deletions.
diff --git a/apps/zui/src/js/state/migrations/202409271055_migrateToSnapshots.ts b/apps/zui/src/js/state/migrations/202409271055_migrateToSnapshots.ts
@@ -78,18 +78,23 @@ export default function migrateToSnapshots(appState: any) {
           if (!snapshot) {
             /* The Snapshot is missing because it has invalid syntax */
             /* But it is stored in the queryVersions */
-            const version = state.queryVersions[queryId].entities[versionId]
+            /* If it is not stored under the queryId, it is stored under the sessionId. */
+            const version =
+              state.queryVersions[queryId].entities[versionId] ||
+              state.queryVersions[sessionId].entities[versionId]
             const isNamed = !!queries[queryId]
-            snapshot = {
-              createdAt: version.ts,
-              updatedAt: version.ts,
-              id: version.version,
-              pins: version.pins,
-              value: version.value,
-              sessionId: sessionId,
-              queryId: isNamed ? queryId : null,
+            if (version) {
+              snapshot = {
+                createdAt: version.ts,
+                updatedAt: version.ts,
+                id: version.version,
+                pins: version.pins,
+                value: version.value,
+                sessionId: sessionId,
+                queryId: isNamed ? queryId : null,
+              }
+              snapshots.push(snapshot)
             }
-            snapshots.push(snapshot)
           }
           /* Re-write the path */
           const newPath = `/snapshots/${snapshot.id}`