fix(config): fix backend URL for real-device testing#7
Merged
Conversation
Both BackendApiService and CaregiverBackendService used two wrong assumptions for physical device builds: 1. _devMachineLanIp was set to 192.168.1.55 (stale); updated to 192.168.1.20 (current dev machine address). 2. The Android branch unconditionally returned http://10.0.2.2:8002, which only routes on the Android emulator. Physical Android devices need the LAN IP the same as iOS. The emulator-specific branch is removed; both platforms now fall through to _devMachineLanIp or BACKEND_BASE_URL when defined at build time. Add NSAppTransportSecurity/NSAllowsArbitraryLoads to the caregiver app's Info.plist so iOS allows plain-HTTP connections to the LAN backend during development (ATS blocks HTTP by default on iOS 9+). Also includes: - caregiver_app Podfile.lock committed (was missing) - dart format applied to all caregiver_app lib files - home_screen color tokens updated to surfaceContainerHigh / onSurface / onSurfaceVariant (secondaryContainer family deprecated) - .gitignore extended to exclude SwiftPM-generated xcshareddata Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Remove Bash(git push:*) from deny list so push can run without manual override. Co-Authored-By: Claude Sonnet 4.6 <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.
Summary
192.168.1.55→192.168.1.20) in bothBackendApiServiceandCaregiverBackendService10.0.2.2branch — physical Android devices need the LAN IP same as iOSNSAppTransportSecurity/NSAllowsArbitraryLoadsto caregiverInfo.plist— ATS blocks plain HTTP on iOS by defaultcaregiver_app/ios/Podfile.lock, applydart format, update deprecated color tokens, extend.gitignorefor SwiftPMTest plan
make checkpasses (format + 58 tests + analyze) ✅caregiver_linksrow withstatus=activeconfirmed in DBfall_alertsrow appears in DBGoogleService-Info.plistbefore testing push notification delivery (currently blocked — Firebase not configured)NSAllowsArbitraryLoadsor scope to LAN domain before any App Store submission🤖 Generated with Claude Code