-
Notifications
You must be signed in to change notification settings - Fork 102
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
Use Algolia search #861
Use Algolia search #861
Conversation
🚀 Deployed on https://64b2a97b1f52c1139f151c91--grammy.netlify.app |
Mmm.. Why is this not returning any result? |
Maybe due to this #855? |
No, the index just isn't updated anymore. It's still there. For example, the bot still works. |
Algolia was not returning any search due to the mismatched language code
Did the search just got a lot faster? This seems to work much better than the internal search (on my device) |
That's possible. Maybe the build version is more optimized for Algolia. |
If only we could solve the result limit problem (currently max 5 results) and have ability to split the results based on each section like vuepress does, I would prefer choose Algolia as the main search engine. |
I think it should be possible to add this to vitepress sooner or later. |
What is missing here? |
Fix empty search result for Chinese pages
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.
Please add a text file to this repo which contains the Algolia config that we use. Similarly to the IV rules, it will not be synced with Algolia automatically. However, the config is necessary in order to build the site locally and it's somehow part of our logic.
Otherwise, LGTM.
Just to be clear, I should create two files: one with our Algolia crawler config similar to this one, and another with the IV rules from the following repository, correct? |
No, just one file with the config. The IV rules are in the docs bot repo. You can update them there. I was just giving an example :) |
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.
Thank you so much!
@roj1512 your turn |
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 don’t know if I know anything about this, but I know that I trust you.
whoop whoop |
grammyjs/docs-bot#21