cphipps/jellyseerr
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix: manual browser refresh would redirect to home on search page (#2692)

Browse Source 
If you used the search feature and tried to manual refresh or share the link, it would reset the
query. Taking you back to the home page.

fix #2683
This commit is contained in:
Brandon Cohen
2022-04-25 06:47:38 -04:00
committed by GitHub
parent b75fc7b238
commit b2878390b4
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/hooks/useSearchInput.ts
View File

@@ -1,9 +1,9 @@
/* eslint-disable react-hooks/exhaustive-deps */ /* eslint-disable react-hooks/exhaustive-deps */
import type { UrlObject } from 'url';
import { useEffect, useState, Dispatch, SetStateAction } from 'react';
import useDebouncedState from './useDebouncedState';
import { useRouter } from 'next/router'; import { useRouter } from 'next/router';
import { Dispatch, SetStateAction, useEffect, useState } from 'react';
import type { UrlObject } from 'url';
import type { Nullable } from '../utils/typeHelpers'; import type { Nullable } from '../utils/typeHelpers';
import useDebouncedState from './useDebouncedState';
type Url = string | UrlObject; type Url = string | UrlObject;
@@ -48,7 +48,7 @@ const useSearchInput = (): SearchObject => {
* in a new route. If we are, then we only replace the history. * in a new route. If we are, then we only replace the history.
*/ */
useEffect(() => { useEffect(() => {
if (debouncedValue !== '') { if (debouncedValue !== '' && searchOpen) {
if (router.pathname.startsWith('/search')) { if (router.pathname.startsWith('/search')) {
router.replace({ router.replace({
pathname: router.pathname, pathname: router.pathname,