-
Notifications
You must be signed in to change notification settings - Fork 76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add methods and classes for Places (New) API in flutter_google_places_sdk_platform_interface
#87
Conversation
flutter_google_places_sdk_platform_interface/analysis_options.yaml
Outdated
Show resolved
Hide resolved
flutter_google_places_sdk_platform_interface/lib/flutter_google_places_sdk_platform.dart
Outdated
Show resolved
Hide resolved
flutter_google_places_sdk_platform_interface/lib/src/types/lat_lng_bounds.dart
Outdated
Show resolved
Hide resolved
flutter_google_places_sdk_platform_interface/lib/src/types/author_attribution.dart
Outdated
Show resolved
Hide resolved
flutter_google_places_sdk_platform_interface/lib/src/types/search_by_text_response.dart
Outdated
Show resolved
Hide resolved
flutter_google_places_sdk_platform_interface/lib/src/types/search_by_text_response.dart
Show resolved
Hide resolved
flutter_google_places_sdk_platform_interface/lib/src/types/search_nearby_response.dart
Outdated
Show resolved
Hide resolved
flutter_google_places_sdk_platform_interface/lib/src/types/search_nearby_response.dart
Show resolved
Hide resolved
hey @flikkr , thanks for the PR! If you can comment on that with some sources from the documentation that would be great - otherwise I would want to do more research before deciding to move. Regardless:
|
Hi @matanshukry, thanks for taking the time to look into this! I've created a sample PR #91 to show how I integrate the full feature in my project. Regarding the concern on moving to the new API, there should not be any breaking changes as the new API can only be used once it is enabled via the Google Cloud console and initialised in app by calling |
@flikkr again sorry for the long delays; |
Signed-off-by: Matan Shukry <[email protected]>
@flikkr I fixed the tests and workflow in master, after resolving conflicts it's now missing the |
Hi @matanshukry I fixed the failing test, please help to run the workflow |
Description
Add classes and methods introduced in Google Places (New) API. I have some changes to add to Android library once this is merged. Tracked by #77.