Skip to content

feat(analyse): review sessions before drilling into laps - #226

Open
Snazzie wants to merge 214 commits into
mainfrom
feat/session-analysis-routes
Open

feat(analyse): review sessions before drilling into laps#226
Snazzie wants to merge 214 commits into
mainfrom
feat/session-analysis-routes

Conversation

@Snazzie

@Snazzie Snazzie commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

Rework Analyse navigation to start with session-level review, then let users drill down into individual laps.

Changes

  • Add Analyse picker for recorded track/car sessions.
  • Show grouped track/car review dashboard before lap analysis.
  • Preserve explicit track, car, laps, and lap route state.
  • Select individual laps from session review to open detailed analysis.
  • Validate incomplete or invalid Analyse selections.
  • Normalize comparison laps to evaluation candidates.
  • Add seeded Playwright coverage for session-to-lap navigation.
  • Add Storybook coverage and canonical snapshots for picker and session review.
  • Update related navigation, chat, import, and review flows.

Result

/analyse opens session review first. Users choose a track/car group, inspect session comparison, then drill into a specific lap.

Korupt-virus and others added 30 commits July 29, 2026 14:50
Base automatically changed from folder-cleanup to main August 11, 2026 19:32
@Snazzie
Snazzie requested a review from a team August 11, 2026 19:32
@Snazzie
Snazzie force-pushed the feat/session-analysis-routes branch from 79d920f to 3705dd5 Compare August 13, 2026 03:30
@blacksmith-sh

blacksmith-sh Bot commented Aug 13, 2026

Copy link
Copy Markdown

Found 1 test failure on Blacksmith runners:

Failure

Test View Logs
src/stories/
dashboards.snapshot.ts:17:3 › snapshot: ComboDash1 ────────────────────────────────
View Logs

Fix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@github-actions github-actions Bot added the ui change Pull request changes rendered dashboard UI label Aug 13, 2026
@Snazzie Snazzie changed the title feat(analyse): add explicit review routes feat(analyse): review sessions before drilling into laps Aug 13, 2026
@Snazzie
Snazzie force-pushed the feat/session-analysis-routes branch from fa4ca2f to 79d920f Compare August 13, 2026 22:04
@github-actions

Copy link
Copy Markdown
Contributor

🖼️ App UI changes

Current PR screenshots compared with main. Storybook render-vs-committed-baseline failures are listed separately from committed PR-baseline-vs-base changes.

Changed: responsive /desktop /fm23-driver

Before After Diff
before after diff

Removed: responsive /desktop /f125-experiment-review

Before After Diff
before after diff

Rendered from Storybook and responsive app screenshot tests against main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui change Pull request changes rendered dashboard UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants