From d4c3b1f4e080ced389f6116980ed8b8fd88d2cbf Mon Sep 17 00:00:00 2001
From: Maurice <mauriceboe@icloud.com>
Date: Mon, 12 Oct 2026 19:57:48 +0000
Subject: [PATCH] refactor(collections): all saved says why it skips a list it
 cannot read

---
 client/src/store/collectionUnion.ts | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/client/src/store/collectionUnion.ts b/client/src/store/collectionUnion.ts
index 58a4b2db9..f6469df85 100644
--- a/client/src/store/collectionUnion.ts
+++ b/client/src/store/collectionUnion.ts
@@ -13,7 +13,13 @@ export async function loadUnionPlaces(ids: number[]): Promise<CollectionPlace[]>
   const seen = new Set<number>();
   for (let i = 0; i < ids.length; i += DETAIL_BATCH) {
     const batch = await Promise.all(
-      ids.slice(i, i + DETAIL_BATCH).map((id) => collectionsApi.get(id).catch(() => null))
+      ids.slice(i, i + DETAIL_BATCH).map((id) =>
+        collectionsApi.get(id).catch(() => {
+          // A list that cannot be read (left, deleted, unreachable) is skipped, so
+          // the union still holds the places of every list that can be.
+          return null;
+        })
+      )
     );
     for (const d of batch) {
       if (!d) continue;