Skip to content
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

Added support of Google Place Autocomplete: #1119

Closed
wants to merge 3 commits into from

Conversation

gdw96
Copy link

@gdw96 gdw96 commented May 6, 2021

  • Add buildPlaceAutocompleteQuery, parseMatchedSubstrings, parseStructuredFormatting and parseTerms methods to GoogleMapsPlaces.php.
  • Add classes GooglePlaceAutocomplete and StructuredFormatting(result of autocompleted queries).
  • Edit fetchURL method for Google Place Autocomplete (response are different).
  • Edit README.md for document the new GeocodeQuery mode.
  • Add testGeocodePlaceAutocompleteMode method to GooglePlaceTest.php. Test pass with success (and tested in a Symfony application successfully).

Hi,

I needed to use Google Place Autocomplete and noticed it wasn't supported, so I set out to add it.

Thank's

 - Add `buildPlaceAutocompleteQuery`, `parseMatchedSubstrings`, `parseStructuredFormatting` and `parseTerms` methods to `GoogleMapsPlaces.php`.
 - Add classes `GooglePlaceAutocomplete` and `StructuredFormatting`(result of autocompleted queries).
 - Edit `fetchURL` method for Google Place Autocomplete (response are different).
 - Edit `README.md` for document the new GeocodeQuery mode.
 - Add `testGeocodePlaceAutocompleteMode` method to `GooglePlaceTest.php`. Test pass with success (and tested in a Symfony application successfully).
@jbelien
Copy link
Member

jbelien commented May 7, 2021

Thanks a lot for your contribution @gdw96 !

First things first, could you make your PR pass the StyleCI test ?

I'll review it ASAP.

@gdw96
Copy link
Author

gdw96 commented May 7, 2021

Hi @jbelien,

Of course, I didn't have time yesterday, I'm correcting it today.

@gdw96
Copy link
Author

gdw96 commented May 7, 2021

@jbelien Done

@Nyholm Nyholm closed this Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants