Add onboarding flow for new users#697
Draft
hthillman wants to merge 11 commits intorafal/graph-mode-frontend-rebasedfrom
Draft
Add onboarding flow for new users#697hthillman wants to merge 11 commits intorafal/graph-mode-frontend-rebasedfrom
hthillman wants to merge 11 commits intorafal/graph-mode-frontend-rebasedfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Comment |
Contributor
🚀 fal.ai Preview Deployment
TestingConnect to this preview deployment by running this on your branch: 🧪 E2E tests will run automatically against this deployment. |
Contributor
✅ E2E Tests passed
Test ArtifactsCheck the workflow run for screenshots. |
6c06866 to
93cda83
Compare
f88bb22 to
27c0870
Compare
…ow support Cherry-picked from 7d1eb0e with conflict resolution. Signed-off-by: Rafał Leszko <rafal@livepeer.org> Signed-off-by: gioelecerati <gioele@cerati.org> Signed-off-by: Hunter Hillman <hthillman@gmail.com> # Conflicts: # frontend/src/components/graph/AddNodeModal.tsx # frontend/src/components/graph/ContextMenu.tsx # frontend/src/components/graph/CustomEdge.tsx # frontend/src/components/graph/GraphEditor.tsx # frontend/src/components/graph/constants.ts # frontend/src/components/graph/nodes/BoolNode.tsx # frontend/src/components/graph/nodes/ControlNode.tsx # frontend/src/components/graph/nodes/ImageNode.tsx # frontend/src/components/graph/nodes/KnobsNode.tsx # frontend/src/components/graph/nodes/MathNode.tsx # frontend/src/components/graph/nodes/MidiNode.tsx # frontend/src/components/graph/nodes/NoteNode.tsx # frontend/src/components/graph/nodes/OutputNode.tsx # frontend/src/components/graph/nodes/PipelineNode.tsx # frontend/src/components/graph/nodes/PrimitiveNode.tsx # frontend/src/components/graph/nodes/RerouteNode.tsx # frontend/src/components/graph/nodes/SinkNode.tsx # frontend/src/components/graph/nodes/SliderNode.tsx # frontend/src/components/graph/nodes/SourceNode.tsx # frontend/src/components/graph/nodes/TupleNode.tsx # frontend/src/components/graph/nodes/VaceNode.tsx # frontend/src/components/graph/nodes/XYPadNode.tsx # frontend/src/components/graph/ui/NodeBody.tsx # frontend/src/components/graph/ui/NodeCard.tsx # frontend/src/components/graph/ui/NodeHeader.tsx # frontend/src/components/graph/ui/NodePillSearchableSelect.tsx # frontend/src/components/graph/ui/NodePillTextarea.tsx # frontend/src/components/graph/ui/index.ts # frontend/src/components/graph/ui/tokens.ts # frontend/src/hooks/useUnifiedWebRTC.ts # frontend/src/hooks/useVideoSource.ts # frontend/src/lib/graphUtils.ts # frontend/src/pages/StreamPage.tsx
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Rafał Leszko <rafal@livepeer.org> Signed-off-by: gioelecerati <gioele@cerati.org> Signed-off-by: Hunter Hillman <hthillman@gmail.com>
Signed-off-by: gioelecerati <gioele@cerati.org> Signed-off-by: Hunter Hillman <hthillman@gmail.com>
Implements the H181 onboarding spec with inference mode selection, cloud auth, workflow picker, and workspace tour. Includes backend persistence endpoints and dual localStorage/API storage layer. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Hunter Hillman <hthillman@gmail.com>
- Fix black thumbnails: use URL strings for public/ assets instead of module imports that Vite can't resolve for public dir files - Delay tour start until first anchor element exists in DOM (polls up to 5s), preventing tour from appearing over blank canvas during workflow import - Remove skipAuth action and UI path — all cloud users must sign in - Refresh cloud status after onboarding auth so the "Remote Inference" toggle reflects the connecting/connected state immediately Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Hunter Hillman <hthillman@gmail.com>
…fix workflow import - Add Manrope as the global app font via Google Fonts + Tailwind config - Add interactive fog-of-war WebGL background to onboarding overlay - Remove workspace tour phase (stub WorkspaceTour, strip tour from context) - Fix workflow import by dismissing overlay before opening import dialog - Update VRAM copy on Run Locally card Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Hunter Hillman <hthillman@gmail.com>
The onboarding overlay creates a second h1 ("Welcome to Daydream Scope")
which causes a strict mode violation in the Playwright locator. Pre-set
localStorage to skip onboarding and use exact heading match.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Hunter Hillman <hthillman@gmail.com>
…ding, Manrope in Electron app - Replace localStorage cache with loading phase + API-only onboarding status - Add FogOfWarBackground + blur veil to Electron ServerLoading screen - Update e2e test to mock onboarding API instead of localStorage - Add Manrope font to all Electron app screens (CSP + Google Fonts link) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Hunter Hillman <hthillman@gmail.com>
…h import - Add cloud_connecting phase between auth and workflow picker — waits for cloud relay to connect before showing workflow selection - CloudConnectingStep triggers activateCloudRelay on mount and shows rotating status messages while waiting - Use sessionStorage to resume onboarding after full-page auth redirect (replaces broken inference_mode-based resume that skipped selection) - Add graph field to all starter workflows so the graph editor renders the pipeline topology after import Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Hunter Hillman <hthillman@gmail.com>
The graph editor didn't pick up the loaded workflow until the user switched modes and back (causing a remount). Call refreshGraph() after handleWorkflowLoad so the graph renders immediately. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Hunter Hillman <hthillman@gmail.com>
…nboarding API - Save workflow graph to localStorage during import so graph editor loads it - Refresh graph editor pipeline list when cloud connects (removes false "not installed" warnings) - Merge locally installed plugin pipeline IDs into available list so plugin pipelines (e.g. yolo_mask) show as available in cloud mode - Make onboarding PUT endpoint accept partial updates (fixes 422 errors) - Add independent polling in CloudConnectingStep to reliably detect connection Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Hunter Hillman <hthillman@gmail.com> # Conflicts: # frontend/src/components/graph/GraphEditor.tsx
31cecf9 to
211c580
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
GET/PUT /api/v1/onboarding/statuswith dual localStorage + API file storage layerdata-tourattribute anchoring system with custom portal-based tour popover (avoids focus-trap conflicts with graph canvas)Remaining TODOs
Test plan
~/.daydream-scope/onboarding.json) sees full onboarding flownpm run build)🤖 Generated with Claude Code