Add InPlay connection strings to the catalog - #140
Merged
Merged
Conversation
The eager connect work (#133) added user-facing strings -- the "Slower Connections Expected" and "Reduced Background Wake-Ups" notice rows in pod settings, and the InPlay connection info screen behind them -- but the regenerated string catalog was never committed, so none of them reached translation. Build-regenerated; no hand edits.
jeremybarnum
pushed a commit
to jeremybarnum/OmnipodKit
that referenced
this pull request
Sep 16, 2026
Lean-out step 1: onto Pete's current next-dev. Ten upstream commits (loopandlearn#119, loopandlearn#125 PodKeepAlive/SilentTune revamp, loopandlearn#133 HostAppState + eager-connect watchdog, loopandlearn#134, loopandlearn#136, loopandlearn#138, loopandlearn#139, loopandlearn#140, loopandlearn#147, loopandlearn#155). Fourteen conflict hunks, resolved to keep his code wherever ours modified the same lines: - lifecycle goes through his HostAppState (its watchOS branch compiles now; our WatchKit import and notification mapping are gone) - his fresh-discovery connect (hoisted out of the DASH branch) replaces ours, which drops our connect-intent guard from that path - his handleDisconnect/iOS-17 variant; our loan disconnect stamp moves into it - his keep-alive tail in shouldHoldConnection, under our watchOS early return - his UIDevice.hasPossibleInPlayBLEIssues and state.podKeepAlive replace our iPhoneWithPossibleInPlayIssues; a watch-only UIDevice stand-in (Common/UIDevice+watchOS.swift) keeps the watch target compiling, since his UIDevice.swift is excluded there - our pod-adoption block in didDiscover and the adoption-marker log in didFailToConnect stay, ahead of his pairing and watchdog code - SilentTune (depends on PumpManagerUI) and silent.wav excluded from the watch target; the audio keep-alive stays behind #if os(iOS) - InPlayConnectionInfoView (his new UI file) excluded from the watch target Co-Authored-By: Claude Fable 5.1 <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
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.
The eager connect work (#133) added user-facing strings — the "Slower Connections Expected" and "Reduced Background Wake-Ups" notice rows in pod settings, and the InPlay connection info screen behind them — but the regenerated string catalog was never committed, so none of them reached translation.
15 new keys, build-regenerated with no hand edits. Spotted by @marionbarker while testing LoopKit/LoopWorkspace#485.