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

Reverse geocoding #110

Open
mikaraunio opened this issue Oct 23, 2022 · 1 comment
Open

Reverse geocoding #110

mikaraunio opened this issue Oct 23, 2022 · 1 comment
Labels

Comments

@mikaraunio
Copy link

With wagtail==4.0.4 and wagtailgeowidget==6.2.0, I have setup linked address and position fields as in the docs, and tested with both Google Maps and Leaflet + Nominatim.

When I change the address, the pin moves on the map as expected.

However, if I move the pin, the address does not update. Is it supposed to, ie. is reverse geocoding supported?

If it isn't, it would be great to have an option to either prevent the user from moving the pin (so the map would just be a 'read-only' confirmation of the address), or to empty the address field if the pin is moved - so as to prevent situations where the address and position are out of sync.

@marteinn
Copy link
Member

marteinn commented Mar 6, 2023

Hi mikaraunio and apologies for the late reply.

However, if I move the pin, the address does not update. Is it supposed to, ie. is reverse geocoding supported?

This is the expected behavior, we do not have reverse geocoding from lat/long to string.
The reason is that I have never seen this requirement, it might also result in a lot of costly requests against the geocoding API.

If it isn't, it would be great to have an option to either prevent the user from moving the pin (so the map would just be a 'read-only' confirmation of the address), or to empty the address field if the pin is moved - so as to prevent situations where the address and position are out of sync.

I think having a read-only map is a good idea, contributions are welcome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants