fix: append a / to the search link. (#432)

aduros · web-flow · commit 60c23d43c5c9 · 2024-07-08T10:52:35.000Z
Most hosting providers HTTP redirect `mysite.com/docs/search?q=test` to `mysite.com/docs/search/?q=test`... except S3 which tosses out the query string and redirects to `mysite.com/docs/search/`.

Rather than relying on the backend redirect to work, tweak the link here on the frontend to already contain the slash suffixed path.
diff --git a/docusaurus-search-local/src/client/theme/SearchBar/SearchBar.tsx b/docusaurus-search-local/src/client/theme/SearchBar/SearchBar.tsx
@@ -220,7 +220,7 @@ export default function SearchBar({
         }
         params.set("version", versionUrl.substring(baseUrl.length));
       }
-      const url = `${baseUrl}search?${params.toString()}`;
+      const url = `${baseUrl}search/?${params.toString()}`;
       a.href = url;
       a.textContent = linkText;
       a.addEventListener("click", (e) => {

Original file line number	Diff line number	Diff line change
`@@ -220,7 +220,7 @@ export default function SearchBar({`
`220`	`220`	`}`
`221`	`221`	`params.set("version", versionUrl.substring(baseUrl.length));`
`222`	`222`	`}`
`223`		- const url = `${baseUrl}search?${params.toString()}`;
	`223`	+ const url = `${baseUrl}search/?${params.toString()}`;
`224`	`224`	`a.href = url;`
`225`	`225`	`a.textContent = linkText;`
`226`	`226`	`a.addEventListener("click", (e) => {`