Properly update state when saving track to Apollo (#445)

GMOD · Sep 20, 2024 · ea5f56e · ea5f56e
1 parent 096c318
commit ea5f56e
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/packages/jbrowse-plugin-apollo/src/session/session.ts b/packages/jbrowse-plugin-apollo/src/session/session.ts
@@ -323,15 +323,15 @@ export function extendSession(
                     if (Object.keys(filteredConfig).length === 0) {
                       filteredConfig = undefined
                     }
-                    let trackConfigSnapshot = getSnapshot(conf) as {
+                    const trackConfigSnapshot = getSnapshot(conf) as {
                       trackId: string
                       type: string
                     }
                     const newTrackId = trackId.slice(
                       0,
                       trackId.length - sessionTrackIdentifier.length,
                     )
-                    trackConfigSnapshot = {
+                    const newTrackConfigSnapshot = {
                       ...trackConfigSnapshot,
                       trackId: newTrackId,
                     }
@@ -346,7 +346,7 @@ export function extendSession(
                           ...filteredConfig,
                           tracks: filteredConfig?.tracks && [
                             ...filteredConfig.tracks,
-                            trackConfigSnapshot,
+                            newTrackConfigSnapshot,
                           ],
                         },
                       })
@@ -357,6 +357,10 @@ export function extendSession(
                       const { notify } = self as unknown as AbstractSessionModel
                       notify('Track added', 'success')
                     }
+                    // @ts-expect-error This method is missing in the JB types
+                    self.deleteTrackConf(conf)
+                    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
+                    jbrowse.addTrackConf(newTrackConfigSnapshot)
                   },
                   icon: SaveIcon,
                 },
@@ -396,6 +400,10 @@ export function extendSession(
                       const { notify } = self as unknown as AbstractSessionModel
                       notify('Track removed', 'success')
                     }
+                    // @ts-expect-error This method is missing in the JB types
+                    self.deleteTrackConf(conf)
+                    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
+                    jbrowse.deleteTrackConf(conf)
                   },
                   icon: SaveIcon,
                 },