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

Update ol-geocoder to ^4.3 to fix search results #197 #198

Merged
merged 1 commit into from Oct 2, 2023

Conversation

paul121
Copy link
Member

@paul121 paul121 commented Oct 2, 2023

Updating ol-geocoder solves the issue and geocoding search works again. There are a couple changes that come with this:

  1. Search results are no longer autocompleted as you type because this was in violation of Nominatim TOS: Search Button - search does not work from smartphone if there is another text field Dominique92/ol-geocoder#255 (comment)
  2. You must press "Enter" or click the new "return" icon to execute the search.

All other existing functionality continues to work the same: click on a result, and map zooms to that location.

Also want to note that the ol-geocoder library has transition maintainers in the past month and many of these changes have been implemented by the new maintainer. Here is the old repo link that redirects to the new: https://github.com/jonataswalker/ol-geocoder

ol-geocoder-updated

Copy link
Collaborator

@symbioquine symbioquine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@paul121 paul121 merged commit 96591cd into farmOS:2.x Oct 2, 2023
2 checks passed
@paul121 paul121 deleted the 2.x-geocoder-search branch October 2, 2023 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants