Add first-run onboarding guide#673
Open
bjorkert wants to merge 4 commits into
Open
Conversation
Introduce a step-based onboarding wizard shown once on launch (skippable, with a prominent Skip for already-configured users): welcome, data source choice, Nightscout/Dexcom connection, units, and a few useful alarms seeded with sensible defaults. For Nightscout, add the option to create a read-only access token from the site's API secret. The secret is used only to authorize the request and is never stored. The token is derived locally from the created subject's id and the secret, since Nightscout's subjects list is served from a cache that does not reflect a freshly created subject right away. Replace the launch-time permission prompts with deferred requests so a fresh install is not fronted with them before onboarding: - notifications are requested when alarms are set up or first added - calendar is requested from the Calendar settings screen - Bluetooth is initialised only when a BLE background refresh mode is selected Welcome screen uses the LoopFollow mark with a coin-landing animation.
Not Looping and Low Battery rely on loop and uploader data that only a Nightscout site provides, so they are meaningless for a Dexcom-only follower. Offer the device/system alarm group during onboarding only when following Nightscout (or a Nightscout URL is already configured), and don't seed them otherwise.
Polish the onboarding wording (data source, Nightscout token, Dexcom sign-in, units, alarms, and completion screens) and modernise the Dexcom field labels: "Username" instead of "User Name" and "Outside US" instead of "NON-US" (display only; stored server value is unchanged). Apply the same Dexcom label changes to the standalone Dexcom settings screen so the two stay consistent.
Member
Author
# Conflicts: # LoopFollow.xcodeproj/project.pbxproj
Collaborator
Collaborator
|
First of all, this is awesome. I played with this yesterday and I really like it. Great first draft! A few general thoughts:
A few more detailed comments about the Onboarding flow
|
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.





















Adds a friendly, step-based first-run onboarding wizard, shown once on launch and skippable (with a prominent Skip for users who are already configured).
Flow
Welcome → choose a data source → connect (Nightscout or Dexcom Share) → units & metrics → a few useful alarms → done.
Highlights
Also modernises the Dexcom field labels ("Username", "Outside US") in both onboarding and the existing Dexcom settings screen for consistency; stored values are unchanged.
A matching docs update will follow separately in the docs repo.