Skip to content

Conversation

tordtst
Copy link
Member

@tordtst tordtst commented Oct 16, 2026

Issue

Closes #50

Issuet lukkes for hånd ved merge til dev. Etter denne PR-en merges dev til main i en egen PR, slik at deployen kan gjøres fra main.

Endringer

  • Ny seksjon «Prøv appen» med VM-adressen, krav om NTNU-nett eller VPN og lenke til lokal kjøring.
  • «Om prototypen» beskriver hva som er nytt i innlevering 2, hva som henter data fra backenden og hva som fortsatt er demo.
  • «Hva som er mockup» sier at annonsene hentes fra databasen. mockListings.ts brukes bare i tester og på designsiden.
  • «Planlagt videre» lister ikke lenger GraphQL og VM, og nevner at API-tester finnes.
  • «Dette ønsker vi tilbakemelding på» har nye spørsmål om oppsettet og samspillet mellom klient og backend.
  • Rettet utdaterte setninger om servervalidering og mockdata.

VM-adressen virker først når frontenden er deployet (egen issue). Den som deployer bør sjekke at adressen stemmer, også http eller https.

Testing

  • npm run lint
  • npm run format:check
  • npm test
  • Lenkene i README er kontrollert i forhåndsvisningen på GitHub

Sjekkliste

  • README eller docs/ er oppdatert hvis det trengs
  • Eventuell KI-bruk er ført i docs/ki-bruk.md

tordtst and others added 3 commits October 16, 2026 19:17
Co-Authored-By: sebasti <sebasti@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: birkjr <birkjr@stud.ntnu.no>
@tordtst tordtst added the documentation Improvements or additions to documentation label Oct 16, 2026
@tordtst tordtst added this to the Innlevering 2 milestone Oct 16, 2026
@tordtst tordtst self-assigned this Oct 16, 2026
@tordtst tordtst requested a review from a team as a code owner October 16, 2026 17:19
@birkjr birkjr merged commit 9dfa1d1 into dev Oct 16, 2026
@andretg andretg deleted the 50-docs-oppdater-readme-for-innlevering-2-og-merge-dev-til-main branch October 18, 2026 19:26
Sign in to join this conversation on GitHub.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants