Merge pull request #255 from DestinyItemManager/saved-search

Whoopsie on saved searches
DestinyItemManager · Dec 18, 2024 · 378c8cb · 378c8cb
2 parents 74e6658 + fe791fd
commit 378c8cb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/api/routes/update.ts b/api/routes/update.ts
@@ -813,7 +813,6 @@ function consolidateSearchUpdates(updates: (UsedSearchUpdate | SavedSearchUpdate
     const u: UpdateSearch = {
       query: group[0].payload.query,
       type: group[0].payload.type ?? SearchType.Item,
-      saved: false,
       incrementUsed: 0,
     };
     for (const update of group) {

diff --git a/api/stately/searches-queries.ts b/api/stately/searches-queries.ts
@@ -119,9 +119,10 @@ export interface UpdateSearch {
   query: string;
   type: SearchType;
   /**
-   * Whether the search should be saved
+   * Whether the search should be saved. If undefined, the saved status is not
+   * changed.
    */
-  saved: boolean;
+  saved?: boolean;
   /** How much to increment the used count by. */
   incrementUsed: number;
 }
@@ -143,7 +144,9 @@ export async function updateSearches(
       existingSearches.find((s) => s.query === query) ??
       newSearch(platformMembershipId, destinyVersion, type, query);
     search.usageCount += incrementUsed;
-    search.saved = saved;
+    if (saved !== undefined) {
+      search.saved = saved;
+    }
     search.lastUsage = BigInt(Date.now());
     return search;
   });