Skip to content

feat: create DriverCard component and update App to display driver data #4

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

feat: create DriverCard component and update App to display driver data #4

merged 1 commit into from
Sep 14, 2026
+100 −18

Conversation

krgjert
Copy link
Member

@krgjert krgjert commented Sep 14, 2026 •
edited
Loading

Screenshot 2026-09-14 at 14 52 06

@krgjert krgjert self-assigned this Sep 14, 2026
@krgjert krgjert requested a review from a team as a code owner September 14, 2026 12:51
wilhelsr
wilhelsr approved these changes Sep 14, 2026
View reviewed changes
Copy link
Member

@wilhelsr wilhelsr left a comment

Choose a reason for hiding this comment

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

Looks good overall. Driver data is fetched and displayed clearly, including the driver name, number and image.

ludvigz
ludvigz approved these changes Sep 14, 2026
View reviewed changes
Copy link
Member

@ludvigz ludvigz left a comment

Choose a reason for hiding this comment

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

Nice cards, good work! We should look into adding information like driver points and position in standing at a later point, but that will require implementeing the championship_drivers endpoint

@krgjert krgjert merged commit 5cbbc99 into dev Sep 14, 2026
@krgjert krgjert deleted the 2-create-driver-component branch September 14, 2026 12:57
Sign in to join this conversation on GitHub.
Reviewers

@wilhelsr wilhelsr wilhelsr approved these changes

@ludvigz ludvigz ludvigz approved these changes

Assignees

@krgjert krgjert

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@krgjert @wilhelsr @ludvigz