Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NAV-24572: Bedre søk på kommuner i barnehagelister #1089

Merged
merged 4 commits into from
Mar 12, 2025

Conversation

bragejahren
Copy link
Contributor

@bragejahren bragejahren commented Mar 10, 2025

NAV-24572

💰 Hva forsøker du å løse i denne PR'en

Tar i bruk Combobox med liste over alle kommuner vi har mottatt barnehagelister fra slik at saksbehandler ikke kan søke på "feil" kommune. Har også refaktorert koden en del, da det var en del unødvendig duplisert / kompliserende kode der. Har også gjort et forsøk på å ta i bruk react-hook-form i filtreringen. Vi bruker nå samme hook for å hente barnehagebarn for både ba-sak og infotrygd.

Funksjonelt skal alt være likt som før, med unntak av kommunenavn-feltet som nå er en Combobox med auto-complete.

Testet OK i preprod.

Relaterte endringer i backend: navikt/familie-ks-sak#1159

🔎️ Er det noe spesielt du ønsker å fremheve?

Synes det er litt rart at man kan filtrere på enten ident eller kommune og ikke begge deler samtidig, men endrer ikke på dette her da det ikke er en del av scope på denne oppgaven.

✅ Checklist

  • Jeg har testet mine endringer i henhold til akseptansekriteriene 🕵️
  • Jeg har skrevet tester.

🤷‍♀ ️Hvor er det lurt å starte?

Alt i ett.

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

👀 Screen shots

Søk på kommune: (se bort fra kommunenavnene, da dette kun er test data fra preprod)
image

@bragejahren bragejahren requested a review from a team as a code owner March 10, 2025 13:44
Copy link
Contributor

@MagnusTonnessen MagnusTonnessen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🌟

bragejahren added a commit to navikt/familie-ks-sak that referenced this pull request Mar 12, 2025
[NAV-24572](https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-24572)

### 💰 Hva skal gjøres, og hvorfor?
Legger til endepunkt for å hente alle kommuner vi har mottatt
barnehagelister for. Dette gir bedre saksbehandlerstøtte i frontend da
listen kan brukes til å hindre at saksbehandler skriver inn feil
kommunenavn ved søk i listene.

Testet OK i preprod.

Relatert endring frontend:
navikt/familie-ks-sak-frontend#1089

### ✅ Checklist
- [ ] Jeg har testet mine endringer i henhold til akseptansekriteriene
🕵️
- [ ] Jeg har config- eller sql-endringer.
- [x] Jeg har skrevet tester.


### 💬 Ønsker du en muntlig gjennomgang?
- [ ] Ja
- [x] Nei
@bragejahren bragejahren merged commit 11ad65e into main Mar 12, 2025
5 checks passed
@bragejahren bragejahren deleted the NAV-24572/bedre-søk-på-kommuner branch March 12, 2025 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants