Skip to content

fix(onboarding): add dashboard escape hatch#3045

Open
Drixled wants to merge 6 commits intomainfrom
fix/get-started-dashboard-link
Open

fix(onboarding): add dashboard escape hatch#3045
Drixled wants to merge 6 commits intomainfrom
fix/get-started-dashboard-link

Conversation

@Drixled
Copy link
Copy Markdown
Contributor

@Drixled Drixled commented May 5, 2026

Summary

  • Adds an authenticated-only Skip to dashboard escape hatch at the bottom of /get-started so signed-in users are not trapped in the starting-point chooser.
  • Keeps the unauthenticated Already have an account? Sign in prompt unchanged while routing authenticated users to /profile, the app's existing dashboard/profile landing route.
  • Architectural changes: none; this updates the existing /get-started client content component only.

Verification

  • No manual browser verification was performed in this session; this was a small conditional link/rendering change and the local app was not opened.
  • Add any additional manual verification details before merge.

Visual Changes

Before After
Screenshot needed: authenticated /get-started without a dashboard escape hatch. Screenshot needed: authenticated /get-started with the Skip to dashboard footer link.

Reviewer Notes

  • The new link is shown only when isAuthenticated is true; unauthenticated users continue to see the existing sign-in prompt.
  • /profile is used because it matches existing dashboard/profile landing conventions elsewhere in the app.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 5, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • apps/web/src/app/get-started/page.tsx
  • apps/web/src/app/get-started/personal/_components/ProductOptionsContent.tsx
  • apps/web/tests/e2e/get-started-dashboard-link.spec.ts

Reviewed by gpt-5.5-2026-04-23 · 606,129 tokens

@Drixled Drixled requested a review from lambertjosh May 5, 2026 16:07
@lambertjosh
Copy link
Copy Markdown
Contributor

Hrm - I'm not seeing it when auth'd?

image

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants