Skip to content

feat: Onboarding Welcome Page#2435

Open
camielvs wants to merge 2 commits into
06-18-feat_onboarding_pillfrom
06-18-feat_onboarding_welcome_page
Open

feat: Onboarding Welcome Page#2435
camielvs wants to merge 2 commits into
06-18-feat_onboarding_pillfrom
06-18-feat_onboarding_welcome_page

Conversation

@camielvs

@camielvs camielvs commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Description

Final PR in the onboarding series (stacked on #2440). Gives new users a dedicated landing experience instead of dropping them into an empty dashboard.

  • Dedicated /welcome route — a focused, centered welcome page hosting the onboarding card, so onboarding doesn't take over the "My Dashboard" view.
  • Index routing (IndexRedirect)/ now resolves based on onboarding state: new users (onboarding not complete and not dismissed) go to /welcome; everyone else goes to /dashboard. The dashboard is moved off / to its own route so manual navigation to either still works.
  • Sidebar "Get started" entry — shown in the dashboard sidebar while onboarding is active, gated on isResolved to avoid flicker.
  • Updates the navigation-tracking e2e spec for the new routes.

Stacked PR — base: Onboarding Pill (#2440), which is stacked on the Onboarding Checklist (#2421). Closes the issue.

Related Issue and Pull requests

Closes https://github.com/Shopify/oasis-frontend/issues/622

Stacked on #2421#2440

Type of Change

  • New feature

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

image.png

Test Instructions

  1. As a fresh user, navigate to / — you should be redirected to /welcome with the onboarding card centered in the available space.
  2. As a user who has completed or dismissed onboarding, navigate to / — you should land on /dashboard.
  3. Manually navigate directly to /welcome and /dashboard — both should be reachable regardless of redirect logic.
  4. While onboarding is active, confirm the Get started item appears in the dashboard sidebar (and does not flash before backend state resolves).
  5. Complete onboarding from /welcome and confirm subsequent visits to / land on the dashboard.

Additional Comments

Routing decisions wait on isReady (queries settled, valid even with no backend) while the sidebar entry gates on isResolved, matching the provider contract introduced earlier in the stack.

@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: 06-18-feat_onboarding_welcome_page/eb185a2

camielvs commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator Author

@camielvs camielvs mentioned this pull request Jun 18, 2026
3 tasks
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch 2 times, most recently from be1cab9 to 475b1de Compare June 19, 2026 01:39
@camielvs camielvs added the #gsd:50583 Learning Hub label Jun 19, 2026 — with Graphite App
@camielvs camielvs changed the base branch from 06-17-feat_onboarding_checklist to graphite-base/2435 June 19, 2026 01:46
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from 475b1de to b2ef001 Compare June 19, 2026 01:46
@camielvs camielvs changed the base branch from graphite-base/2435 to 06-18-feat_onboarding_pill June 19, 2026 01:46
@camielvs camielvs mentioned this pull request Jun 19, 2026
3 tasks
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from 33d4a5c to c38d713 Compare June 19, 2026 02:09
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from b2ef001 to 46b33cb Compare June 19, 2026 02:09
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from c38d713 to 5f5d39b Compare June 19, 2026 02:25
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch 2 times, most recently from 4833c25 to a93cf71 Compare June 19, 2026 15:43
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from 5f5d39b to c994062 Compare June 19, 2026 15:43
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from a93cf71 to e0ce7f1 Compare June 19, 2026 15:44
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch 2 times, most recently from bbdf96e to 5ea2aaa Compare June 19, 2026 15:51
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from e0ce7f1 to 8c45c84 Compare June 19, 2026 15:51
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from 90f6e8b to 8cdc3b4 Compare June 26, 2026 17:20
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from 9daead8 to 25a3375 Compare June 26, 2026 17:20
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from 8cdc3b4 to e396a7e Compare June 26, 2026 17:22
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from 25a3375 to 274bf7f Compare June 26, 2026 17:22
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from e396a7e to 9e53492 Compare June 26, 2026 19:28
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from 274bf7f to 9096703 Compare June 26, 2026 19:28
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from 9e53492 to dc15573 Compare June 26, 2026 19:49
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch 2 times, most recently from 733a175 to 9dea09e Compare June 26, 2026 20:52
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch 2 times, most recently from 74401d4 to 75d35ae Compare June 26, 2026 21:10
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from 9dea09e to 2b92751 Compare June 26, 2026 21:10
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from 75d35ae to 2a27cb9 Compare June 26, 2026 21:13
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from 2b92751 to 2404644 Compare June 26, 2026 21:13
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from 2a27cb9 to d0c8019 Compare June 26, 2026 21:28
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from 2404644 to 714eabd Compare June 26, 2026 21:28
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from d0c8019 to c47e06f Compare June 26, 2026 21:29
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from 714eabd to 50a2381 Compare June 26, 2026 21:29
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from c47e06f to b04af07 Compare June 26, 2026 21:30
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch from 50a2381 to 23e92c7 Compare June 26, 2026 21:30
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch from b04af07 to 6e26b09 Compare June 26, 2026 21:50
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch 2 times, most recently from 1a96436 to 6a73580 Compare June 26, 2026 22:00
@camielvs camielvs force-pushed the 06-18-feat_onboarding_welcome_page branch 2 times, most recently from 0203fac to 1e17352 Compare June 26, 2026 23:01
@camielvs camielvs force-pushed the 06-18-feat_onboarding_pill branch 2 times, most recently from 193c890 to c296a9f Compare June 26, 2026 23:25
camielvs and others added 2 commits June 26, 2026 17:07
- Gate OnboardingWelcome on !isReady with a spinner (match IndexRedirect)
- Import APP_ROUTES from appRoutes instead of the router shim

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

#gsd:50583 Learning Hub

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants