Skip to content

Conversation

andretg
Copy link
Member

@andretg andretg commented Oct 30, 2026

Issue

Closes #69

Endringer

  • «Om prototypen» beskriver nå innlevering 3. Innlevering 1 og 2 er samlet i en kort tabell, og det som er nytt i innlevering 3 står som en liste med lenker til dokumentasjonen.
  • «Planlagt videre» har ikke lenger punkter som er ferdige (søk på serveren, tilgjengelighet, bærekraft og Playwright). Nytt punkt om det som gjenstår til innlevering 4: test med skjermleser, utypisk bruk og snapshot-tester.
  • Ny del, «Valg og begrunnelser»:
    • Søk, filtre og skalering: søk og filtre i databasen, indekser, limit/offset, debouncing, cache og hvordan vi unngår N+1, med lenker til målingene i docs/datasett.md.
    • State management: hvorfor tilstanden ligger i Apollo-cachen, URL-en, sessionStorage og lokal React-state, og hvorfor vi ikke bruker Redux.
    • Komponenter og biblioteker: hva hvert bibliotek brukes til og hvorfor, og hvorfor vi ikke bruker et komponentbibliotek.
  • «Dette ønsker vi tilbakemelding på» har nye, konkrete spørsmål om søk og filtre, tilgjengelighet, bærekraft, kodekvalitet og oppsett.
  • Lenken til appen på VM-en bruker http://, siden Apache ikke er satt opp med HTTPS.
  • Tabellen over visninger nevner alle filtrene i katalogen.

Verdt å se på: begrunnelsene i «Valg og begrunnelser», særlig om komponentbibliotek og pg uten ORM. Stemmer de med hvorfor vi faktisk valgte det slik?

Testing

Bare README.md er endret, så koden er ikke påvirket.

  • npm run format:check (kjørt som npx prettier --check --end-of-line auto README.md på Windows)
  • Alle lenker og ankere i README er sjekket og peker til filer og overskrifter som finnes
  • Noen i gruppen som ikke har skrevet endringene, har fulgt README for å kjøre appen fra et nytt klone

Sjekkliste

  • README eller docs/ er oppdatert hvis det trengs

- Beskriv innlevering 3 i «Om prototypen» og fjern utdaterte punkter i «Planlagt videre»
- Legg til «Valg og begrunnelser» om skalering, state management og biblioteker
- Nye spørsmål til medstudentene om søk, tilgjengelighet, bærekraft og kodekvalitet
- Bruk http i lenken til appen på VM-en

Refs #69

Co-authored-by: Sebastian Ingebrigtsen <sebastianingebrigtsen@users.noreply.github.com>
@andretg andretg added this to the Innlevering 3 milestone Oct 30, 2026
@andretg andretg requested a review from a team as a code owner October 30, 2026 10:11
Copy link
Member

@birkjr birkjr left a comment

Choose a reason for hiding this comment

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

Ser bra ut! README er mer oversiktlig nå, og det er fint at endringene for innlevering 3 og valgene vi har tatt er bedre dokumentert. Approved

@birkjr birkjr merged commit d760811 into dev Oct 30, 2026
@tordtst tordtst deleted the 69-docs-oppdater-readme-for-innlevering-3 branch October 30, 2026 13:26
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants