-
Notifications
You must be signed in to change notification settings - Fork 77
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
Add searchbar to docs #858
Conversation
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.
Thanks for adding this! It immediately makes the docs way more useful and accessible.
Have one small comment re: a config value, but otherwise LGTM.
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.
actually... some comments need to be addressed first
Codecov Report
@@ Coverage Diff @@
## main #858 +/- ##
=======================================
Coverage 64.44% 64.44%
=======================================
Files 107 107
Lines 9283 9283
=======================================
Hits 5982 5982
Misses 3301 3301 Continue to review full report at Codecov.
|
🎨 Hid the see more text as the see more page is currently broken 🔀 Merged with main
docs/web/docusaurus.config.js
Outdated
searchParameters: {}, | ||
|
||
// Optional: path for search page that enabled by default (`false` to disable it) | ||
searchPagePath: "search", |
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.
can we also bring back this config and set it to false
?
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 set searchPagePath to false
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.
ty!
Resolves #709
Login to algolia here:
https://www.algolia.com/users/sign_in
You can see search plugin here:
https://www.algolia.com/dashboard
To trigger a manual crawl(website gets automatically crawled once every week) of the website login here:
https://crawler.algolia.com/
The button to click for a manual crawl
Double Search Result Issue (RESOLVED)
exclusionPatterns: ["https://mephisto.ai//*/**"],
Video
search.mov
Issue to be resolved later:
Video of the bug (I unhid the "see all results" text for this video)
video-with-error.mov