fix: carry over multiple params (#1252)

#1118 Fixes issue with carry over logic not bringing for params that are defined multiple times
chanzuckerberg · Oct 18, 2024 · 3c86411 · 3c86411
1 parent 3312884
commit 3c86411
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/frontend/packages/data-portal/app/utils/url.ts b/frontend/packages/data-portal/app/utils/url.ts
@@ -50,11 +50,7 @@ export function carryOverFilterParams({
   prevParams: URLSearchParams
 }) {
   for (const key of filters) {
-    const value = prevParams.get(key)
-
-    if (value) {
-      params.set(key, value)
-    }
+    prevParams.getAll(key).forEach((value) => params.append(key, value))
   }
 
   return params