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

Discoverer v2 support. Spelling suggestions feature #7

Merged
merged 5 commits into from
Nov 6, 2024

Conversation

chrispenny
Copy link
Contributor

@chrispenny chrispenny commented Oct 29, 2024

For the changes being proposed in Discoverer:
silverstripeltd/silverstripe-discoverer#21

And Discoverer > Elastic Enterprise:
silverstripeltd/silverstripe-discoverer-elastic-enterprise#9

Config change

No practical change here other than updating the injection for the adaptors, but it just felt like it was getting a bit long, and probably deserved some categorisation through separate yaml files.

Breaking changes (summary)

  • Discoverer v2
  • Configuration name discoverer-bifrost has been split out into multiple yaml files: discoverer-bifrost-adaptors, discoverer-bifrost-requests, and discoverer-bifrost-factory
  • Argument names in Request classes have been changed (EG see Service/Requests/Search:: __construct())

@chrispenny chrispenny force-pushed the feature/spelling-suggestion branch 2 times, most recently from cf4ef9e to 4cb0d58 Compare October 31, 2024 19:33
@chrispenny chrispenny marked this pull request as ready for review October 31, 2024 19:33
@chrispenny chrispenny force-pushed the feature/spelling-suggestion branch from 4cb0d58 to 20d23c9 Compare October 31, 2024 21:10
@chrispenny chrispenny force-pushed the feature/spelling-suggestion branch from 0d3ee4e to 866e3cf Compare November 3, 2024 22:10
Copy link
Contributor

@blueo blueo left a comment

Choose a reason for hiding this comment

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

lgtm - working well in local tests with the search ui module

@chrispenny chrispenny merged commit 89ee60e into main Nov 6, 2024
20 checks passed
@chrispenny chrispenny deleted the feature/spelling-suggestion branch November 6, 2024 01:10
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.

2 participants