Update onboarding docs for new Welcome/Model/Access/Ready flow - #6
Open
ronaldmannak wants to merge 1 commit into
Open
Update onboarding docs for new Welcome/Model/Access/Ready flow#6ronaldmannak wants to merge 1 commit into
ronaldmannak wants to merge 1 commit into
Conversation
The first-run flow was reworked in PicoServer (#38, #42, #48): it is now a four-step Welcome -> Model -> Access -> Ready wizard that picks a first model (RAM-based recommendation + background download) and sets network access, rather than a toggle list including Open-at-login / Prevent-sleep. Correct the Install and Run page to match the shipped flow and note the menu extra's Memory and Models-in-Memory sections. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0171A1mZB4sL46Wy2pbwDJCu
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
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
Keeps the Install and Run getting-started page in sync with the onboarding rework merged in
ronaldmannak/PicoServersince the last documentation run. All facts are grounded in the current source (OnboardingWizardView.swift,MenuExtraView.swift), not point-in-time PR descriptions.What changed
This Mac onlyvsDevices on my local network, with Bonjour discovery as a nested option shown only when LAN access is on. Updated the LAN / local-only steps to name these choices.Open at loginandPrevent sleep. The reduced flow (#42) no longer exposes those; the page now notes they live in the native app settings.Memorysection (Apps / MLX / Free bar + pressure indicator) andModels in Memorylist.Evidence
Verified against
ronaldmannak/PicoServermainat the #49 merge (9088e48): the wizard'sWizardPageset (welcome/models/access/ready),AccessPage's two options + nested Bonjour toggle,ReadyPage's status/model/access/address rows, andMenuExtraView'sMemoryandModels in Memorysections.Not done on purpose
inferenceProviderisnil, so the Inference section never renders for end users (the source comment: "Release builds keep this nil so the Inference section never shows fabricated numbers"). Documenting a DEBUG-only, placeholder-data section would be inaccurate; deferred until it ships in release.Relationship to other open documentation PRs
Allow notificationstoggle that no longer exists — notification permission is now requested implicitly at first server start, not via an onboarding toggle. This PR supersedes that onboarding content; Document onboarding notifications toggle and menu-extra memory section #5's onboarding portion should be closed or reconciled, and its memory-section content is subsumed here.🤖 Generated with Claude Code
Generated by Claude Code