Skip to content

Api - #40

Merged
Yasminjama merged 9 commits into
mainfrom
api
Oct 2, 2026
Merged

Api#40
Yasminjama merged 9 commits into
mainfrom
api

Conversation

@Thebeatkicks

Copy link
Copy Markdown
Owner

Sammanfattning

Beskriv kort vad denna pull request ändrar och varför.

Ändringar

  • Konvertera det ni redan har. api.js → api.ts, guidevyn och guidekortet med lang="ts" – exakt som i demon. En fil per commit, så att diffen går att läsa. Tours-vyn får vara JS tills vidare (allowJs är på).
  • Typa en route i API:et. guides.js → guides.ts med Response<Guide[]>. Kolla att npm run dev:api fortfarande startar – Node kör filen direkt.
  • uppdaterade types i shared

Testning

Beskriv hur ändringarna testades, eller skriv varför testning inte behövdes.

  • Testat lokalt
  • Körda tester: npm test (om projektet har tester)
  • Kontrollerat relevanta manuella flöden

Checklista

  • Koden följer projektets befintliga struktur och stil
  • Onödiga debug-utskrifter och kommentarer är borttagna
  • Dokumentation har uppdaterats om det behövs
  • PR:n är redo för review

Review

  • Minst en teammedlem har reviewat ändringarna
  • Review-kommentarer är hanterade
  • PR:n är godkänd innan merge

@Yasminjama
Yasminjama self-requested a review October 2, 2026 08:48
@Yasminjama
Yasminjama merged commit 629eb88 into main Oct 2, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants