fix: prevent double encode if we are on /search endpoint

sct · Jan 13, 2023 · 252fdca · 252fdca
1 parent c2a1a20
commit 252fdca
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/hooks/useDiscover.ts b/src/hooks/useDiscover.ts
@@ -46,16 +46,21 @@ const useDiscover = <
         return null;
       }
 
+      const paramsEncodeCheck = (
+        params: Record<string, unknown>,
+        paramKey: string
+      ) =>
+        endpoint.includes('/search')
+          ? params[paramKey]
+          : encodeURIExtraParams(params[paramKey] as string);
+
       const params: Record<string, unknown> = {
         page: pageIndex + 1,
         ...options,
       };
 
       const finalQueryString = Object.keys(params)
-        .map(
-          (paramKey) =>
-            `${paramKey}=${encodeURIExtraParams(params[paramKey] as string)}`
-        )
+        .map((paramKey) => `${paramKey}=${paramsEncodeCheck(params, paramKey)}`)
         .join('&');
 
       return `${endpoint}?${finalQueryString}`;