feat: add dedicated tire outings dashboard page#308
feat: add dedicated tire outings dashboard page#308Anton15K wants to merge 2 commits intoslowlydev:mainfrom
Conversation
- implement `OutingTable` component for session-wide tire history - create dedicated `/dashboard/tires` page - enhance `DriverHistoryTires` with stint progression and used set markers - integrate "Tires" link into sidebar navigation - update help documentation with tire history details
There was a problem hiding this comment.
Pull request overview
Adds a dedicated dashboard page for session-wide tire stint (“outing”) history, addressing Issue #293 and exposing tire history in the UI/navigation and help docs.
Changes:
- Added
OutingTablecomponent to display per-driver tire stint history across the session. - Created new
/dashboard/tirespage and added a “Tires” entry to the dashboard sidebar. - Enhanced tire history rendering and updated Help documentation to point to the new page.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| dashboard/tsconfig.json | Switches TS module resolution to bundler for the dashboard package. |
| dashboard/src/components/driver/DriverHistoryTires.tsx | Updates stint rendering (key change + used-set marker display). |
| dashboard/src/components/dashboard/OutingTable.tsx | New session-wide tire outing table UI component. |
| dashboard/src/components/Sidebar.tsx | Adds “Tires” link to dashboard navigation. |
| dashboard/src/app/dashboard/tires/page.tsx | New Tires dashboard route rendering OutingTable. |
| dashboard/src/app/(nav)/help/page.tsx | Adds Help note linking to the Tires dashboard page. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <p className="whitespace-nowrap text-sm font-medium leading-none text-zinc-400"> | ||
| {stint.TotalLaps}L{stint.New === "FALSE" ? "*" : ""} | ||
| </p> |
There was a problem hiding this comment.
Stint.New is typed as a string and the help example models a used set by omitting New (only setting New: "TRUE" for new tires). With the current check stint.New === "FALSE", stints where New is undefined (likely used sets) won’t get the used-set marker. Consider aligning this logic with the existing DriverTire behavior and treating anything other than an explicit "TRUE" as used (or otherwise normalize/clarify the data contract).
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
implementing feature from issue #293
OutingTablecomponent for session-wide tire history/dashboard/tirespageDriverHistoryTireswith stint progression and used set markers