Skip to content

Conversation

mathealk
Copy link
Member

Hva er endret?

Legger til søkefunksjonalitet i landlisten. Søkefeltet er koblet til GraphQL-spørringen countries(search: ...), og søket gjøres i databasen (ILIKE, så det skiller ikke mellom store og små bokstaver).

  • Ny "useDebounce"-hook (300 ms), så det ikke sendes en forespørsel per tastetrykk
  • "SearchBar" er nå en kontrollert komponent ("value"/"onChange")
  • "GET_COUNTRIES" tar imot $search og sender den til backend
  • Last-, feil- og tomtilstand vises under søkefeltet, så søkefeltet ikke forsvinner mens det lastes. Ved null treff vises «No countries found.»

Ikke inkludert: paginering/«last inn mer» (listen er fortsatt begrenset til 20 treff), sortering og filtrering. Backend er uendret, siden "search" allerede ble støttet i resolveren.

Type endring

  • Frontend
  • Backend / GraphQL
  • Database
  • Konfig / verktøy
  • Dokumentasjon
  • Bug / fix

Test endringene

  • Kjør npm run lint for å sjekke kodekvalitet med ESLint
  • Kjør npm run format:check for å sjekke formatering med Prettier
  • Kjør npm run build for å kontrollere TypeScript og produksjonsbygget
  • Test endringene manuelt i appen

Manuell testing

Forutsetninger: koblet til NTNU VPN, backend/.env er fylt ut.

  1. Åpne SSH-tunnel: ssh -N -L 5432:localhost:5432 <brukernavn>@it2810-28.idi.ntnu.no
  2. Start backend (cd backend && npm run dev) og frontend (npm run dev i rotmappen), og åpne http://localhost:5173/
  3. Skriv «nor» i søkefeltet. Listen skal filtreres til land med «nor» i navnet. Skriv «NOR» og kontroller at du får samme treff.
  4. Skriv raskt og kontroller i Network-fanen at det sendes én forespørsel ca. 300 ms etter siste tastetrykk, ikke én per tegn.
  5. Søk på noe uten treff (f.eks. «zzz») og kontroller at «No countries found.» vises.
  6. Klikk på et land i resultatlisten og kontroller at detaljsiden vises. «Back» skal ta deg tilbake til listen.

Sjekkliste

  • Endringene er testet, eller eventuelle begrensninger er beskrevet.
  • Dokumentasjonen er oppdatert dersom det er nødvendig.
  • Ingen utilsiktede endringer er inkludert.

Closes #13

@mathealk mathealk requested a review from a team as a code owner October 13, 2026 15:31
@mathealk mathealk linked an issue Oct 13, 2026 that may be closed by this pull request
4 tasks
Copy link
Member

@guroklo guroklo left a comment

Choose a reason for hiding this comment

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

Testet søkefunksjonaliteten manuelt. Søk fungerer uavhengig av store og små bokstaver, debounce ser ut til å fungere som forventet, og meldingen ved null treff vises riktig. Navigasjon til landdetaljer og tilbake fungerer også. Ingen feil oppdaget under manuell testing.

@guroklo guroklo merged commit 478a212 into main Oct 14, 2026
@guroklo guroklo deleted the 13-søkefunksjonalitet branch October 14, 2026 09:14
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] Implementer søkefunksjonalitet
2 participants