Skip to content

Conversation

sebasti
Copy link
Member

@sebasti sebasti commented Oct 29, 2026

Issue

Closes #62

Endringer

  • Ny tabell reviews med fremmednøkkel til listings (ON DELETE CASCADE), CHECK på score 1–5 og kommentar ≤ 500 tegn, og indeks på (listing_id, created_at DESC, id DESC).
  • Listing har fått averageScore, reviewCount og reviews(limit: Int = 10). Snitt og antall regnes ut med CROSS JOIN LATERAL bare for annonsene som returneres, uten N+1.
  • createReview(input) validerer score og kommentar på serveren og svarer med vurderingen og annonsen med oppdatert snitt og liste. Ukjent annonse gir «Fant ikke annonsen.»
  • Annonsekortet og detaljvisningen viser snittet som stjerner og tekst («4,7 av 5 (3 vurderinger)»). Detaljvisningen har de 10 nyeste vurderingene og skjemaet «Gi en vurdering».
  • Den nye vurderingen vises med en gang: mutasjonen og detaljspørringen bruker samme fragment, så Apollo oppdaterer detaljvisningen og kortet fra cachen.
  • 9 eksempelvurderinger i npm run seed.
  • Egen commit: gjør en katalogtest fra [FEAT] Koble søkefeltet i katalogen til serveren #57 mindre avhengig av tid.

Verdt å se på: docs/vurderinger.md beskriver valgene, og hva som mangler uten innlogging.

Testing

  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npm test (frontend 283, backend 161 mot testdatabase)
  • npm run build
  • Kontrollert i nettleser mot backend med 50 010 annonser, også på 375 px og med tastatur

Nye tester: reviews.api.test.ts, createReviewInput.test.ts, ReviewForm, ReviewList, ReviewScore, reviewForm.test.ts, reviewFormat.test.ts, og vurderinger i App.test.tsx.

Migrasjonen er kjørt opp, ned og opp igjen lokalt. På VM-en kjøres npm run migrate (og eventuelt npm run seed) i #68.

Sjekkliste

  • README eller docs/ er oppdatert hvis det trengs

@sebasti sebasti requested a review from a team as a code owner October 29, 2026 21:17
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 ryddig ut! Bra at både validering og tester er på plass, og fint at gjennomsnittet oppdateres direkte etter en vurdering.

@birkjr birkjr merged commit 1d022ca into dev Oct 30, 2026
@tordtst tordtst deleted the 62-feat-legg-til-vurderinger-av-annonser 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