Repository navigation
fix(copy): correct stale and misleading WHOOP 5.0/MG text on Apple, Android and docs - #2795
Open
JohanStefanes wants to merge 1 commit into
Open
JohanStefanes wants to merge 1 commit into
JohanStefanes wants to merge 1 commit into
Conversation
…ndroid and docs User-facing text only; no BLE, gating, analytics or storage behaviour changes. - Point every 5/MG probe reference at Test Centre -> 5/MG protocol diagnostics -> Protocol probes (or Connection & Sync), not the removed Settings -> Experimental section (Refs ryanbr#2464) - Strap alarm: say on both platforms that a 5/MG only arms while Protocol probes is on, and that an earlier alarm may still be set on the strap - Probe verdicts for opcodes 98/84 say "answered", not "ACCEPTED", identically on Swift and Kotlin (tests updated) - Health sync card, Sync now hint and Shortcuts reply no longer claim a pairing handshake is in progress for a hello-suppressed 5/MG - Pairing-mode step says how (tap until the LEDs flash blue); onboarding expectation card, steps, skin-temp, SpO2 and HRV caveats match the code - Translations updated in all Android locales and the xcstrings catalog - README, FEATURES, FAQ, ANDROID, ANALYTICS and protocol docs match the app Verified: macOS (Strand) and iOS (NOOPiOS) builds succeed; WhoopProtocol and StrandAnalytics swift test pass; Android testFullDebugUnitTest passes except two RecoveryDriversTest cases that fail identically on main; i18n_audit --ci and doc_comment_lint pass. Not tested on hardware (text only). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
Text-only pass over user-facing WHOOP 5.0/MG copy on Apple (macOS/iOS), Android and the docs. Every changed string was first checked against the code that decides the behaviour; no BLE command, gate, analytics formula or stored value changes.
Main corrections:
Localizable.xcstrings. The MG ECG medical disclaimer stays deliberately untranslated, as before.Historical changelog entries are untouched; only the live onboarding Expectation card in
AppChangelog.swiftchanged.Verification
xcodebuildscheme Strand (macOS) and NOOPiOS (iOS Simulator): build succeeded.swift test: WhoopProtocol (816 tests) and StrandAnalytics (2143 tests) pass../gradlew testFullDebugUnitTest: passes apart from 2RecoveryDriversTestcases (half-tie rounding) that fail the same way on unmodified main.python3 -m unittest discover -s Tools: passes apart from 4test_check_source_referencescases (/varvs/private/vartemp-path on macOS) that fail the same way on unmodified main.Tools/i18n_audit.py --ciandTools/doc_comment_lint.py: pass.StrandTestswas not run locally;app-build.ymlruns it on this PR.🤖 Generated with Claude Code