-
Notifications
You must be signed in to change notification settings - Fork 3k
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
feat(docs): Improve documentation about Search #5889
feat(docs): Improve documentation about Search #5889
Conversation
(cherry picked from commit ec016b4)
Adding some verbiage to usage section
Adding some changes to the results section
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.
I made various changes directly as commits on this PR. Overall looking good!
Once @gabe-lyons 's advanced search features land, we'll need to come back and revisit this
@szalai1 @jjoyce0510 we get a lot of questions from OSS about how Search results are ordered - can we add in those details in FAQ? |
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.
This is looking great!! Left a few comments/requested changes for you to consider
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.
Look great!
(cherry picked from commit ec016b4)
Checklist