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

Smart auto suggest #1154

Draft
wants to merge 79 commits into
base: main
Choose a base branch
from
Draft

Smart auto suggest #1154

wants to merge 79 commits into from

Conversation

edsu
Copy link
Contributor

@edsu edsu commented Aug 12, 2024

This PR adjusts the auto suggestions to provide three categories of results: locations, datasets and maps. It updates the suggest endpoint to draw from two suggestion dictionaries.

closes #1047

thatbudakguy and others added 30 commits July 29, 2024 21:08
use bootstrap.bundle instead of bootstrap
update header to stanford header
Now that geoblacklight/geoblacklight#1441 has
been merged we should probably be using main?
dnoneill and others added 25 commits August 12, 2024 17:21
Update to the latest version of the component library
small header icons fixes search page
move search header into document split
Use container-fluid and set max-width to 1600px
Point at CDN versions of stylesheets for dependencies
- return more 25 suggestions instead of 5 to possibly get a better mix
  of datasets and maps when using the titleSuggest dictionary.
- remove the commented out search based suggest
- move the splitting of datasets and maps out of the template and into the response class
- switch the spatialSuggester from FuzzyLookupFactory to
  BlendedInfixLookupFactory to get the match highlighting
- link results to search URLs (is there a better way to do this?)
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.

7 participants