Conversation
"Set to my entered address" does not work when only country is selected. The country field uses country codes (ex: nl,uk,de etc). The google maps api does not know how to convert those. My suggestion would be to use the text of the selected option and send that to the google maps api. This seems to work better.
|
I cannot reproduce this problem. On the contrary, country codes work better than full names: Code yields direct hit on the country: While full name yields some irrelevant addresses: |
|
I mentioned Google Maps Api (assumption) was used. But openstreetmap.org is used when only a country is used as parameter. Api calls for openstreetmap: The google maps api call does indeed not understand "Nederland" instead of "nl". Should we send both parameters to mod_geo notify and pass different queries to googlemaps or openstreetmap? |
|
Yes, that seems the best way; mod_geo already has access to all address parameters. Can you change this PR? |
…eld works for both services
…eld works for both services
|
@ddeboer Changed the pull request so both services have their own set of parameters. |
|
Looks good to me. @arjan Do you agree? If so, can you squash and merge this? |
|
Merged this at https://github.com/zotonic/mod_geo. |
"Set to my entered address" does not work when only country is selected. The country field uses country codes (ex: nl,uk,de etc). The google maps api does not know how to convert those. My suggestion would be to use the text of the selected option and send that to the google maps api. This seems to work better.