Skip to content

Conversation

guroklo
Copy link
Member

@guroklo guroklo commented Oct 14, 2026 •

Hva er endret?

Endringene inkluderer:

  • Oppdatert beskrivelse av implementert og planlagt funksjonalitet.
  • Beskrivelse av prosjektets arkitektur og skalerbarhet.
  • Dokumentasjon av frontend, GraphQL-backend og PostgreSQL-database.
  • Oppdatert informasjon om hvordan backend kjøres på NTNU VM.
  • Beskrivelse av hvordan KI har blitt brukt under utviklingen.
  • Fjernet eller oppdatert utdatert informasjon.

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

Sjekkliste

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

Closes #36

@guroklo guroklo requested a review from a team as a code owner October 14, 2026 18:52
@guroklo guroklo linked an issue Oct 14, 2026 that may be closed by this pull request
4 tasks
Copy link
Member

@ellahw ellahw left a comment

Choose a reason for hiding this comment

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

Ser veldig bra ut. Fant bare et par småting, kommenterte de fleste i git men nevner de her:))

  • "GraphØL" i implemented so far
  • ".env. local" skal være .env.local
  • setningen "npm run dev. The backend does not need..." mangler starten, "Then run" e.l.
  • "Further functionality is planned..." gjentar det som står i Planned Functionality rett under, så den kan kanskje fjernes
  • kanskje legge til en setning øverst i About the Project som nevner byene, eks: "The dataset also includes more than 34,000 cities linked to their countries, so the solution can be tested on a larger dataset."
  • veldig småpirk, men noen overskrifter har fet skrift, mens andre ikke. kanskje bare fjerne ekstra fet skrift på de det gjelder ettersom de får tykkere skrift automatisk når det er overskrifer?

Copy link
Member

@mariehd mariehd left a comment

Choose a reason for hiding this comment

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

Bra jobba:) Se kommentarer for endringer for å få den godkjent:)

Copy link
Member

@ellahw ellahw left a comment

Choose a reason for hiding this comment

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

Alle kommentarene er fikset og det ser bra ut nå! Eneste er at format:check feiler på README-filen, så kjør gjerne npm run format en siste gang før merge. Bra jobba!

Copy link
Member

@mariehd mariehd left a comment

Choose a reason for hiding this comment

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

Nydelig, veldig godkjent!:)

Copy link
Member

@ellahw ellahw left a comment

Choose a reason for hiding this comment

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

Perfekt nå!

@ellahw ellahw merged commit 3453810 into main Oct 16, 2026
@ellahw ellahw deleted the 36-docs-oppdater-readme branch October 16, 2026 13: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.

[docs] Oppdater README
3 participants