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

Suggestions #32

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Suggestions #32

wants to merge 15 commits into from

Conversation

krahulreddy
Copy link
Member

@krahulreddy krahulreddy commented Aug 29, 2020

These code changes are made as part of GSoC 2020. This Pull Request contains changes made to Nominatim UI in order to provide suggestions to Nominatim DB.

The repository nominatim-indexing can be used to set up suggestions. This repository explains how to set up suggestions and how to use a hug API to create an endpoint to access these suggestions.

Once the elasticsearch indexing is complete, You can use the nominatim-ui to access these suggestions.

The following important things are to be set up in order for the suggestions to be accessible:

  • In dist/config.js, Suggestions_Url needs to be updated with the url of your suggestions end point.
  • The languages and other important name tags need to be updated on dist/assets/js/suggest.js.

The same changes mentioned here can be used to set up suggestions on any website. Few things like icon links need to be changed in suggest.js.

File changes description:

  • config.example.js: Added the suggestions endpoint as one of the parameters.
  • search.html: search bar now uses suggest.js to provide suggestions.
  • suggest.js: Makes the network calls on input and fetches the results. This script is responsible for providing suggestions.
  • suggest.css: Few classes used to display the suggestions as an Unordered list with icons.

@krahulreddy krahulreddy marked this pull request as ready for review August 31, 2020 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant