From 9287e7b93c3510abda156ad482ce42d52f8301a3 Mon Sep 17 00:00:00 2001
From: keyan <keyan.kousha+huumn@gmail.com>
Date: Wed, 18 May 2022 16:09:44 -0500
Subject: [PATCH] allow user to visit/bookmark empty search page

---
 components/seo.js  | 4 ++--
 fragments/items.js | 2 +-
 pages/search.js    | 9 +++++----
 3 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/components/seo.js b/components/seo.js
index bdfed6bc..57959875 100644
--- a/components/seo.js
+++ b/components/seo.js
@@ -5,8 +5,8 @@ import RemoveMarkdown from 'remove-markdown'
 export function SeoSearch ({ sub }) {
   const router = useRouter()
   const subStr = sub ? ` ~${sub}` : ''
-  const title = `${router.query.q} \\ stacker news${subStr}`
-  const desc = `SN${subStr} search: ${router.query.q}`
+  const title = `${router.query.q || 'search'} \\ stacker news${subStr}`
+  const desc = `SN${subStr} search: ${router.query.q || ''}`
 
   return (
     <NextSeo
diff --git a/fragments/items.js b/fragments/items.js
index da31a7d8..23459c22 100644
--- a/fragments/items.js
+++ b/fragments/items.js
@@ -112,7 +112,7 @@ export const ITEM_WITH_COMMENTS = gql`
 
 export const ITEM_SEARCH = gql`
   ${ITEM_FIELDS}
-  query Search($q: String!, $cursor: String) {
+  query Search($q: String, $cursor: String) {
     search(q: $q, cursor: $cursor) {
       cursor
       items {
diff --git a/pages/search.js b/pages/search.js
index 10ecf27b..8ae388b3 100644
--- a/pages/search.js
+++ b/pages/search.js
@@ -5,7 +5,7 @@ import SearchItems from '../components/search-items'
 import { useRouter } from 'next/router'
 import { SeoSearch } from '../components/seo'
 
-export const getServerSideProps = getGetServerSideProps(ITEM_SEARCH, null, null, 'q')
+export const getServerSideProps = getGetServerSideProps(ITEM_SEARCH)
 
 export default function Index ({ data: { search: { items, cursor } } }) {
   const router = useRouter()
@@ -13,9 +13,10 @@ export default function Index ({ data: { search: { items, cursor } } }) {
   return (
     <Layout noSeo>
       <SeoSearch />
-      <SearchItems
-        items={items} cursor={cursor} variables={{ q: router.query?.q }}
-      />
+      {router.query?.q &&
+        <SearchItems
+          items={items} cursor={cursor} variables={{ q: router.query?.q }}
+        />}
     </Layout>
   )
 }