Skip to content

Håndtert loading- og error-state for sangtekst #46

Jump to bottom
Merged
merged 1 commit into from
Sep 16, 2026
Merged

Håndtert loading- og error-state for sangtekst #46

merged 1 commit into from
Sep 16, 2026
+2 −2

Conversation

ellahw
Copy link
Member

@ellahw ellahw commented Sep 12, 2026

Endringer

  • Hentet ut isLoading og error fra useLyrics i SongCard
  • Viser lasteindikator mens sangteksten hentes
  • Viser tydelig feilmelding hvis sangen mangler tekst i API-et eller kallet feiler

Test

  • npm run lint fungerer uten feil
  • npm run build fungerer uten feil
  • Åpne appen med npm run dev og:
    • Test at lasteindikator vises mens teksten hentes (kan simuleres med "Slow 3G" i devtools)
    • Test at feilmelding vises for sanger uten tekst i API-et

Closes #18

@ellahw ellahw requested a review from a team as a code owner September 12, 2026 15:16
@ellahw ellahw linked an issue Sep 12, 2026 that may be closed by this pull request
Håndter loading- og error-state #18
Closed
2 tasks
mariehd
mariehd approved these changes Sep 12, 2026
View reviewed changes
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.

Funker som det skal, nais!

@mariehd mariehd merged commit 9a95f09 into main Sep 16, 2026
Sign in to join this conversation on GitHub.
Reviewers

@mariehd mariehd mariehd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Håndter loading- og error-state
2 participants
@ellahw @mariehd