Merge pull request #1060 from lpsinger/search-params-default-page

Don't add page to search params when on default page (page 1)
nasa-gcn · Jun 26, 2023 · 5e3e1b0 · 5e3e1b0
2 parents dcfdad4 + 8402819
commit 5e3e1b0
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/app/routes/circulars/index.tsx b/app/routes/circulars/index.tsx
@@ -68,9 +68,9 @@ function getPageLink(
   startDate?: string,
   endDate?: string
 ) {
-  const searchParams = new URLSearchParams({ page: page.toString() || '1' })
+  const searchParams = new URLSearchParams()
+  if (page > 1) searchParams.set('page', page.toString())
   if (query) searchParams.set('query', query)
-  searchParams.set('page', page.toString() || '1')
   if (startDate) searchParams.set('startDate', startDate)
   if (endDate) searchParams.set('endDate', endDate)
   return `?${searchParams.toString()}`