-
Notifications
You must be signed in to change notification settings - Fork 25
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(config): add Algolia search configuration to docusaurus.config.js #66
feat(config): add Algolia search configuration to docusaurus.config.js #66
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.
Very good jobs for me, thanks @mimisavage, @metalboyrick can check?
@metalboyrick pelase what is your telegram username so i can share the env details ? |
@mimisavage it's the same name as the github. you should be able to do it here (you can give it any name you like, e.g. |
I did this in my recent commit here f77fe00 |
@mimisavage please wait, I will soon make corresponding changes to the CI so that it will unblock this PR |
No more comments on my end! |
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.
overall looks good, CI PR will be merged after ODHack ends. Seems like we might experience some hiccups while it is not merged yet since server is not set yet. cc @Nadai2010 @0xquantum3labs
Thanks for the PR @mimisavage ! |
considering that this PR may break the CI, will be holding it back until other OD PRs are merged. |
Okay sir. Thank you |
gm @mimisavage do you have tg that we can connect with? |
Task name here
Fixes #61
Types of change
Comments (optional)