Support the systemSmall widget family in SundialWidget #126
+374
−139
Conversation
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
…s to be shown in watchOS complication configurations
Widgets were failing when snapshot() and timeline() were called in quick succession because CLGeocoder rate limits reverse geocoding requests. - Add LocationAppGroupCache to share location and placemark data between the main app and widget extensions via App Group UserDefaults - Main app now caches placemark (title, subtitle, timezone) alongside location - Widget reads cached placemark from App Group, avoiding redundant geocoding - Widget only geocodes as fallback when no cached data or location moved >500m - Geocoder instance moved inside actor to prevent concurrent calls Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Reduces duplication by replacing four nearly-identical structs with a single shared LocationData struct that conforms to Codable and AnyLocation. Replaced: - LocationAppGroupCache.CachedPlacemark - SavedLocationEntityData (in LocationAppEntity) - TemporaryLocationData (in LocationAppEntity) - SavedLocation.CodableRepresentation (now a typealias) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Removes the hasPlacemarkData guard that prevented reading cached location when placemark fields were nil. The widget will trigger geocoding if timeZoneIdentifier is missing, maintaining the separation between time-based location cache and distance-based placemark cache. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…r-timeline-issue
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.
Closes #122