-
Notifications
You must be signed in to change notification settings - Fork 185
Rationalize solutions/search nav #3753
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
Conversation
🔍 Preview links for changed docs |
| The accompanying [GitHub repository](https://www.github.com/elastic/elasticsearch-labs) contains hands-on Python notebooks and sample apps to help you get started with these advanced search features. | ||
| ::: | ||
|
|
||
| ## Generative AI tools |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unrelated to nav, minor updates on this page
benironside
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No comment on the ToC reorg, but left a couple suggestions about the feature descriptions for your consideration.
@benironside thanks for the copyedits! But on the review process: seeing as the ToC reorg is the important part of the PR here, it doesn't make sense to approve the PR on your side if you're not reviewing that :) |
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
|
LGTM, this improvement simplifies the ToC a great deal. I especially like having full-text, AI-powered, hybrid search, and ranking and reranking under search approaches. |
@georgewallace based on our discussion yesterday, tried to clean up the nav here and avoid inconsistencies in the search approaches
Screenshots
🍏
🔴
