-
Notifications
You must be signed in to change notification settings - Fork 45
chore: add site verification #589
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
Conversation
Signed-off-by: Sertac Ozercan <sozercan@gmail.com>
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.
Pull Request Overview
This PR adds Algolia site verification metadata to the Docusaurus configuration to enable Algolia's crawler to index the website.
- Adds a meta tag with
algolia-site-verification
for domain verification - Integrates with existing Algolia search configuration to complete the indexing setup
@@ -98,6 +98,17 @@ const config = { | |||
apiKey: 'e2cfa004b0a812062660e0039aca0bda', | |||
indexName: 'kaito-aikit', | |||
}, | |||
headTags: [ | |||
{ | |||
tagName: "meta", |
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.
[nitpick] Use single quotes for consistency with the existing codebase. The file uses single quotes for strings (lines 98, 99), but this line uses double quotes.
tagName: "meta", | |
tagName: 'meta', |
Copilot uses AI. Check for mistakes.
attributes: { | ||
// Allow Algolia crawler to index the site | ||
// See https://www.algolia.com/doc/tools/crawler/getting-started/create-crawler/#verify-your-domain. | ||
name: "algolia-site-verification", |
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.
[nitpick] Use single quotes for consistency with the existing codebase. The file uses single quotes for strings (lines 98, 99), but this line uses double quotes.
name: "algolia-site-verification", | |
name: 'algolia-site-verification', |
Copilot uses AI. Check for mistakes.
// Allow Algolia crawler to index the site | ||
// See https://www.algolia.com/doc/tools/crawler/getting-started/create-crawler/#verify-your-domain. | ||
name: "algolia-site-verification", | ||
content: "58101301D914B63C", |
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.
[nitpick] Use single quotes for consistency with the existing codebase. The file uses single quotes for strings (lines 98, 99), but this line uses double quotes.
content: "58101301D914B63C", | |
content: '58101301D914B63C', |
Copilot uses AI. Check for mistakes.
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, using
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when the PR gets merged):Fixes #
Special notes for your reviewer: