Marketing screenshot infra + ThresholdsView extraction#4
Closed
PrernaWeb wants to merge 3 commits into
Closed
Conversation
Moves the ~130-line thresholds block (run pace / FTP / swim CSS / threshold HR, the confirm flow, and the stale-set re-estimate nudge) out of the large SettingsView into its own view. Its body is a Section, so it renders identically whether embedded in Settings' Form or shown on its own. Pure, behaviour-preserving refactor; SettingsView now calls ThresholdsView(model:). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
DEBUG/COACHSAGE_DEMO-gated only: - DemoMode: seed a full A/B/C season and 12 weeks of progressHistory (with realistic easy-session heart rates so the sRPE-based CTL/ATL land in a credible range rather than elite figures). - HomeModel.refreshProgress: use the seeded history in demo mode so the Progress screen renders without HealthKit data. - RunCoachApp: add progress/season/thresholds COACHSAGE_SCREEN routes. The progress route owns its HomeModel via @StateObject so the async history load survives a demo-root re-render (inline models were captured empty). - capture script: add the three new screens. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- WidgetMockupView + COACHSAGE_SCREEN=widget route: renders the medium Home Screen widget on a wallpaper for marketing capture (re-created for capture like LiveActivityScreenshotView, since a widget extension can't be screenshotted from the app). - RunCoachWatchApp: seed a demo WidgetSnapshot into the App-Group store under COACHSAGE_DEMO so the standalone watch app renders real content on a watchOS sim (no phone to sync from). DEBUG-only. - capture script: add the widget screen. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Owner
Author
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
1 participant
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.
Supporting app-side changes behind the CoachSage website refresh (the site content itself already shipped to `main`). Everything here is DEBUG/`COACHSAGE_DEMO`-gated except the ThresholdsView extraction, which is a behaviour-preserving refactor.
Commits
Notes
🤖 Generated with Claude Code