Skip to content

Conversation

mathealk
Copy link
Member

Hva er endret?

Oppdaterte readme, la til forklarende kommentarer i koden hvor det ikke er innlysende hva koden gjør.

Type endring

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

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.
  • Kjør npm test for å kjøre automatiserte tester med Vitest.
  • Start appen med npm run dev og test endringene manuelt i appen.

Manuell testing

Ikke relevant

Sjekkliste

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

Closes #

@mathealk mathealk requested a review from a team as a code owner October 30, 2026 14:20
@mathealk mathealk linked an issue Oct 30, 2026 that may be closed by this pull request
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.

README-en ser veldig bra ut og er ryddig og informativ! Fant bare noen småting som kan oppdateres:

  • Under Planned Functionality står sortering, filtrering og Visited/Want to visit fortsatt som planlagt, selv om dette allerede er implementert.
  • Under Technology står det Vitest (coming), men vi har allerede automatiserte tester.
  • WCAG-arbeidet kan flyttes fra planlagt til implementert funksjonalitet, siden vi nå har gjennomført tilgjengelighetstesting med blant annet VoiceOver og Lighthouse.
  • Under Other Commands kan vi legge til npm test, slik at det er tydelig hvordan man kjører testene.
    Ellers ser dokumentasjonen veldig bra ut!

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.

Dritbra!!!

@guroklo guroklo merged commit 415f2c1 into main Oct 30, 2026
1 check passed
@guroklo guroklo deleted the 37-docs-oppdater-readme-sprint-3 branch October 30, 2026 14:52
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[docs] Oppdater README (Sprint 3)
2 participants