[google_maps_flutter] Add a pinned-SDK web example app#11761
[google_maps_flutter] Add a pinned-SDK web example app#11761stuartmorgan-g wants to merge 9 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the google_maps_flutter_web package to address the removal of heatmap support in version 3.65 of the Google Maps JavaScript SDK. Key changes include updating the README with deprecation notices and instructions for pinning the SDK to version 3.64, as well as restructuring the example apps into latest and 3-64 directories. Heatmap integration tests have been moved to the pinned 3-64 example. Review feedback includes corrections for a typo in the documentation, fixing inaccurate comments in the test files, and removing unused dependencies in the new example's configuration.
|
@bparrishMines @tarrinneal If either of you have a chance to review this before @mdebbar please take a look, since it will let us re-open the tree. |
The tree is currently closed because Google Maps SDK 3.65 came out, and it throws exceptions why trying to use heatmaps, as that functionality has been removed from the SDK.
This fixes the tree by creating a second example app for
google_maps_flutter_webthat pins version 3.64 of the Google Maps SDK:The heatmap tests have been removed from the main example, where they throw.
The README has been updated to explain that heatmaps are deprecated, and using them requires pinning 3.64.
Fixes flutter/flutter#186946
Pre-Review Checklist
[shared_preferences]///).Footnotes
Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling. ↩ ↩2