Fjerne unødvendig sjekk som hindrer søk på kriterier

navikt · Oct 11, 2024 · 6c19e02 · 6c19e02
1 parent 346d730
commit 6c19e02
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 16 deletions.
diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
@@ -45,7 +45,7 @@ jobs:
     deploy-til-dev:
         name: Deploy til dev-gcp
         needs: bygg-og-push-docker-image
-        if: github.event_name == 'workflow_dispatch' || github.ref == 'refs/heads/master' || github.ref == 'refs/heads/fjerne-filterOrd'
+        if: github.event_name == 'workflow_dispatch' || github.ref == 'refs/heads/master' || github.ref == 'refs/heads/fjerne-sjekk'
         runs-on: ubuntu-latest
         steps:
             - uses: actions/checkout@v3

diff --git a/src/kandidatsok/hooks/useSøkekriterierFraStilling.ts b/src/kandidatsok/hooks/useSøkekriterierFraStilling.ts
@@ -1,23 +1,21 @@
-import { useEffect, useState } from 'react';
-import { useSearchParams } from 'react-router-dom';
-import { FilterParam } from './useQuery';
-import useSøkekriterier, { LISTEPARAMETER_SEPARATOR } from './useSøkekriterier';
-import { KandidatsokQueryParam } from 'felles/lenker';
-import { HentFylkerDTO, useHentFylker } from '../../api/stillings-api/hentFylker';
 import {
     formaterStedsnavn,
     lagKandidatsøkstreng,
     stedmappingFraGammeltNavn,
     stedmappingFraGammeltNummer,
 } from 'felles/MappingSted';
 import { Rekrutteringsbistandstilling } from 'felles/domene/stilling/Stilling';
+import { useEffect, useState } from 'react';
+import { HentFylkerDTO, useHentFylker } from '../../api/stillings-api/hentFylker';
+import { FilterParam } from './useQuery';
+import useSøkekriterier, { LISTEPARAMETER_SEPARATOR } from './useSøkekriterier';
 
 const useSøkekriterierFraStilling = (
     rekrutteringsbistandstilling: Rekrutteringsbistandstilling | undefined,
     brukKriterierFraStillingen: boolean
 ) => {
     const { setSearchParam } = useSøkekriterier();
-    const [searchParams] = useSearchParams();
+    // const [searchParams] = useSearchParams();
     const [harLagtTilKriterier, setHarLagtTilKriterier] = useState(false);
 
     const { data: fylker, isLoading: fylkerIsLoading } = useHentFylker();
@@ -40,14 +38,19 @@ const useSøkekriterierFraStilling = (
         if (
             rekrutteringsbistandstilling &&
             brukKriterierFraStillingen &&
-            søkeKriterierIkkeLagtTil(searchParams) &&
             !fylkerIsLoading &&
             !harLagtTilKriterier
         ) {
             anvendSøkekriterier(rekrutteringsbistandstilling);
         }
-        // eslint-disable-next-line react-hooks/exhaustive-deps
-    }, [rekrutteringsbistandstilling, brukKriterierFraStillingen, JSON.stringify(fylker)]);
+    }, [
+        rekrutteringsbistandstilling,
+        brukKriterierFraStillingen,
+        fylkerIsLoading,
+        harLagtTilKriterier,
+        fylker,
+        setSearchParam,
+    ]);
 };
 
 const hentØnsketYrkeFraStilling = (rekrutteringsbistandstilling: Rekrutteringsbistandstilling) => {
@@ -92,9 +95,4 @@ const hentØnsketStedFraStilling = (
     }
 };
 
-const søkeKriterierIkkeLagtTil = (searchParams: URLSearchParams) =>
-    Array.from(searchParams.keys()).every(
-        (param) => param === KandidatsokQueryParam.Kandidatliste
-    ) || Array.from(searchParams.keys()).every((param) => param === KandidatsokQueryParam.Stilling);
-
 export default useSøkekriterierFraStilling;