Merge pull request #353 from OpenGeoscience/remove-todo

annehaley · web-flow · commit c1c133c1e1cd · 2026-03-19T10:37:37.000-04:00
Remove todo statement (see issue #143)
diff --git a/web/src/store/layer.ts b/web/src/store/layer.ts
@@ -180,7 +180,7 @@ export const useLayerStore = defineStore('layer', () => {
           }
         }
 
-        // TODO: Move this conditional functionality into `addLayer`, and directly call addLayerFrameToMap there
+        // Add current frame to map if not added yet
         const sourceId = mapStore.sourceIdFromLayerFrame(layer, frame);
         if (layer.visible && !userMapLayers.some(
           (mapLayerId) => mapLayerId.includes(sourceId)

Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,7 @@ export const useLayerStore = defineStore('layer', () => {`
`180`	`180`	`}`
`181`	`181`	`}`
`182`	`182`
`183`		- // TODO: Move this conditional functionality into `addLayer`, and directly call addLayerFrameToMap there
	`183`	`+ // Add current frame to map if not added yet`
`184`	`184`	`const sourceId = mapStore.sourceIdFromLayerFrame(layer, frame);`
`185`	`185`	`if (layer.visible && !userMapLayers.some(`
`186`	`186`	`(mapLayerId) => mapLayerId.includes(sourceId)`