Skip to content

Conversation

hasanmo
Copy link
Member

@hasanmo hasanmo commented Sep 16, 2026

Change

Denne PR-en samler og tar i bruk felles TypeScript-modeller for byer og aktiviteter:

  • Flytter modellene fra type/ til types/.
  • Bruker én felles liste med åtte byer.
  • Bruker City, CityId, Activity og ActivityId på tvers av appen.
  • Legger til sykling som aktivitet.
  • Oppdaterer aktivitetsvalg, lagring og værpoengberegning.
  • Fjerner dupliserte modeller og data.

Denne PR-en bør merges til dev før PR-en for responsivt design. Design-branchen må deretter hente siste versjon av dev, slik at CSS- og komponentendringene bygges på de nye modellene og aktivitetsdataene.

Verification

Kjørt lokalt:

npm run typecheck
npm run lint
npm run test
npm run build

Manuelt kontrollert:

  • Alle åtte byer vises.
  • Byvelgeren fungerer.
  • Aktivitetsvelgeren viser alle aktivitetene.
  • Valgt aktivitet lagres og brukes i værberegningen.
  • Værdata vises for byene.

Course requirements and AI use

Endringen oppfyller kravet om TypeScript-modeller uten bruk av any og gjør modellene tilgjengelige for både komponenter og databehandling.

KI ble brukt til å diskutere struktur, feilsøke TypeScript-feil og foreslå integrasjonen. Koden og funksjonaliteten er kontrollert lokalt av gruppen.

Review

Jeg ber om at en medstudent gjennomgår PR-en før merge. Sjekk spesielt modellene, aktivitetsvelgeren og at eksisterende værvisning fortsatt fungerer.

@hasanmo hasanmo requested a review from a team as a code owner September 16, 2026 23:46
@olestan
Copy link
Member

olestan commented Sep 17, 2026

Enig! Jeg tar responsivt design-branchen etter at denne er merget til dev, og henter siste versjon av dev før jeg fortsetter der.

Copy link
Member

@olestan olestan 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! Jeg har sett gjennom endringene i by- og aktivitetsmodellene, aktivitetsvelgeren og værberegningen. Alt ser greit ut, så jeg godkjenner PR-en.

@olestan olestan merged commit 11985f9 into dev Sep 17, 2026
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