Skip to content

Conversation

clemenbe
Copy link
Member

delte opp App.tsx i komponenter. det lå tre nesten helt like sections der som gjorde akkurat det samme, så nå er det:

  • Section - tar tittel + laster/feil-melding, innholdet sendes inn som children
  • MovieList - viser filmene som MovieCard
  • PersonList - viser skuespillerne

la til types/tmdb.ts med Movie, Person og TmdbResponse så vi slipper any overalt. typene går nå fra api-laget gjennom hookene og ut i komponentene av seg selv, så vi får feilene i editoren i stedet for i nettleseren.

i api/tmdb.ts var fetch-funksjonene identiske bortsett fra url og feilmelding, så de deler nå en felles tmdbFetch. skal vi endre headers eller legge til noe senere er det ett sted i stedet for tre.

merget inn main underveis og fikk med MovieCard fra Leona. den er urørt og brukes nå i alle filmseksjonene. MovieList har jeg derimot endret fra å hente data selv til å ta imot filmer som props - den gamle varianten kalte usePopMovies internt og viste derfor alltid populære filmer, så den kunne ikke gjenbrukes til "Nye filmer" eller søkeresultater senere. kommentarene hennes er beholdt, bortsett fra de som beskrev fetching som ikke skjer i den fila lenger.

ryddet også bort død kode fra da vi byttet mellom faner (viewConfig, gammel view-state) og et par filer som bare lå igjen.

søket er ikke med her - Sidebar har en valgfri onSearch-prop klar til den som tar den biten.

eslint og tsc er grønne. filmene vises som kort nå, men de har ingen CSS enda så de ser rå ut - det er neste jobb.

@clemenbe clemenbe requested a review from a team as a code owner September 10, 2026 13:37
@laspark laspark merged commit 9b29776 into main Sep 10, 2026
@clemenbe clemenbe deleted the component-sidebar-main-fix branch September 18, 2026 08:34
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