feat: add Algolia DocSearch configuration#1380
feat: add Algolia DocSearch configuration#1380Abhash-Chakraborty wants to merge 4 commits intorecodehive:mainfrom
Conversation
|
@Abhash-Chakraborty is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
There was a problem hiding this comment.
Pull request overview
This PR adds optional Algolia DocSearch support to the Docusaurus site so search can be enabled cleanly once the required Algolia credentials are available, without breaking builds or showing a nonfunctional navbar search entry.
Changes:
- Read Algolia DocSearch environment variables in
docusaurus.config.tsand enable search only when the full config is present. - Conditionally add the navbar search item and Algolia theme configuration to avoid partial/misconfigured setup at runtime.
- Document the required Algolia environment variables in
.env.example, including the need to use the real crawler-created index name.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
package-lock.json |
Lockfile metadata churn from dependency resolution / merge updates. |
docusaurus.config.ts |
Adds conditional Algolia env parsing, warning behavior for partial config, navbar search item wiring, and theme-level Algolia config. |
.env.example |
Documents the new optional Algolia DocSearch environment variables and index-name guidance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@sanjay-kv any updates on this one ? |
|
@sanjay-kv , can you attach the code given in the agolia documentation for implementation |
@Adez017 app id has to replace with env variables, i have changed it in the vercel env files |
| apiKey: algoliaSearchApiKey!, | ||
| indexName: algoliaIndexName!, | ||
| contextualSearch: true, | ||
| }, |
There was a problem hiding this comment.
hi @Abhash-Chakraborty can you clarify about this :: the agolia haven't been configured correctly
|
@sanjay-kv i dont think the script has been configured correctly in the PR , if the changes are already been there @Abhash-Chakraborty can help with any updates on this . |
|
✅ Synchronized metadata from Issue #44:
|


Summary
ALGOLIA_APP_ID,ALGOLIA_SEARCH_API_KEY, andALGOLIA_INDEX_NAMEALGOLIA_INDEX_NAMEmust be the real Algolia index name created by the crawlerupstream/maininto this branchVerification
npm run build✅npm run lint✅ (warnings only)npm run typecheck❌ existing repository-wide TypeScript errors unrelated to this branchNotes
Index ... does not existerror should clear once DocSearch indexing is completed andALGOLIA_INDEX_NAMEpoints to the actual Algolia index name rather than a crawler/config id.Closes: #44