fix(place_tracker): Migrate to Dart 3 and resolve Null Safety crashes#2794
fix(place_tracker): Migrate to Dart 3 and resolve Null Safety crashes#2794rabi-33 wants to merge 4 commits intoflutter:mainfrom
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
|
i signed it |
|
@googlebot I signed it |
|
Thanks @rabi-33 - Looks like the changes don't pass CI. I think you just need to add |
|
@ericwindmill Done! I've added "resolution: workspace" to the place_tracker/pubspec.yaml. Thanks for the catch! |
|
@ericwindmill Hey Eric, just a gentle ping on this! The CI checks are all green and the workspace resolution has been added. Let me know if there's anything else you need from my side to get this merged. Thanks! |
Problem
The
place_trackersample was pinned to an outdated SDK version (>=2.7.0 <3.0.0) and used pre-Null Safety dependencies. This caused the following issues:Maps_flutterversion (0.5.x) utilized deprecated Android embedding v1 APIs.Solution
This PR migrates the project to fully support Dart 3 and Flutter 3.x.
Key Changes:
sdk: ">=3.0.0 <4.0.0".Maps_flutter: Upgraded to^2.5.0(fixes Android embedding warning).provider: Upgraded to^6.0.0.go_router: Added for modern navigation handling.?nullable types andlatekeywords throughout the codebase.Null check operator used on a null valueerrors.Verification
flutter analyzewith 0 errors.Pre-launch Checklist
///).