Cameronmorrow1/fence 2540 update flutter to 3234#90
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the Flutter Radar SDK from version 3.12.4 to 3.23.4, bringing several new features and platform compatibility updates.
Key Changes
- Adds new API methods for product management, tags, verified tracking, and in-app messaging
- Updates RadarSDK dependencies across iOS (3.23.4) and Android (3.23.4) platforms
- Increases minimum iOS deployment target from 10.0 to 15.6
- Enhances
trackVerifiedandsearchPlacesmethods with additional parameters
Reviewed Changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| lib/flutter_radar.dart | Adds new API methods (setProduct, getTags, addTags, removeTags, isTrackingVerified, clearVerifiedLocationToken, showInAppMessage), enhances trackVerified and searchPlaces with new parameters, adds useMotion/usePressure to tracking presets |
| ios/flutter_radar.podspec | Updates RadarSDK version to 3.23.4 and iOS platform requirement to 15.6 |
| ios/flutter_radar.podspec.template | Updates iOS platform requirement template to 15.6 |
| ios/Classes/RadarFlutterPlugin.m | Implements new method handlers for iOS platform, updates import to use @import, adds parameters to trackVerified and searchPlaces implementations |
| example/pubspec.yaml | Updates permission_handler dependency from 10.2.0 to 12.0.0 |
| example/lib/main.dart | Adds showInAppMessage demo, updates foregroundService options to use iconString instead of icon |
| example/ios/Runner.xcodeproj/xcshareddata/xcschemes/Runner.xcscheme | Adds LLDB debugger configuration |
| example/ios/Runner.xcodeproj/project.pbxproj | Updates iOS deployment target to 15.6 |
| example/ios/Podfile | Updates platform to iOS 15.6 and RadarSDKMotion to 3.23.4 |
| example/ios/Flutter/AppFrameworkInfo.plist | Updates MinimumOSVersion from 12.0 to 13.0 |
| example/android/settings.gradle | Updates Android Gradle Plugin from 7.4.2 to 8.6.0 |
| example/android/gradle/wrapper/gradle-wrapper.properties | Updates Gradle from 7.5 to 8.7 |
| example/android/app/src/main/java/io/radar/example/MainApplication.java | Removes deprecated FlutterMain initialization |
| example/android/app/build.gradle | Adds namespace, updates compileSdkVersion to 35, updates RadarSDK to 3.23.4 |
| android/src/main/java/io/radar/flutter/RadarFlutterPlugin.java | Implements new method handlers for Android platform, updates Radar.initialize call with additional parameters, adds parameters to trackVerified and searchPlaces implementations |
| android/gradle/wrapper/gradle-wrapper.properties | Updates Gradle wrapper from 7.0.2 to 8.5 |
| android/build.gradle | Adds Kotlin plugin dependency and configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
| s.dependency 'RadarSDK', '3.19.5' | ||
| s.platform = :ios, '10.0' | ||
| s.dependency 'RadarSDK', '3.23.4' | ||
| s.platform = :ios, '15.6' |
There was a problem hiding this comment.
why was it needed to bump the iOS version all the way to 15.6?
There was a problem hiding this comment.
the iOS SDK doesn't have this requirement, why does the flutter package need to specify a higher version?
https://github.com/radarlabs/radar-sdk-ios/blob/master/RadarSDK.podspec
There was a problem hiding this comment.
It's some quirk with the flutter compiler, the new swift async code in the iam class causes an app crash if the iOS platform target is set below 15.
There was a problem hiding this comment.
Would it be possible to find some workarounds to not use the new async techniques and use a legacy approach?
This is a mayor change disguised on a hot fix version without announcing the breaking changes. Without any warning we're forced to stay on a previous release until we get internal approval to increase the minimum deployment target of the app.
There was a problem hiding this comment.
You're right, this should have been released as a beta, apologies for the inconvenience. We'll look into reversing the iOS platform target.
preliminary sdk update to v3.23.4. I will be thoroughly upgrading all the sdks in the near future so I think we should release this as just a beta for now