merge: #4006

4006: Do not allow the same asset to be added to a list more than once r=jobelenus a=jobelenus <img src="https://media1.giphy.com/media/mc7uduXz800wtqU7WV/giphy.gif"/> Co-authored-by: John Obelenus <[email protected]>
systeminit · Jun 20, 2024 · 13d7215 · 13d7215
2 parents fcc3d48 + 489e796
commit 13d7215
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/app/web/src/store/asset.store.ts b/app/web/src/store/asset.store.ts
@@ -186,10 +186,12 @@ export const useAssetStore = () => {
       },
       actions: {
         addAssetSelection(id: AssetId) {
-          this.selectedAssets.push(id);
-          this.LOAD_ASSET(id);
-          this.syncSelectionIntoUrl();
-          this.selectedFuncs = [];
+          if (!this.selectedAssets.includes(id)) {
+            this.selectedAssets.push(id);
+            this.LOAD_ASSET(id);
+            this.syncSelectionIntoUrl();
+            this.selectedFuncs = [];
+          }
         },
         setAssetSelection(id: AssetId) {
           if (!this.selectedAssets.includes(id)) {