chore: add ahrefs verification#1276
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughThis PR adds an Ahrefs site verification meta tag to the site root component. The meta tag is conditionally included in the document head only when rendering the home page, using an existing ChangesHome page Ahrefs verification
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR adds an Ahrefs domain verification meta tag to the Docusaurus root layout so the site can be verified with Ahrefs.
Changes:
- Injects an
ahrefs-site-verification<meta>tag into the document<head>on the home page.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/theme/Root.tsx`:
- Line 178: The JSX conditional rendering on the isHome expression has stray
whitespace after the self-closing meta tag which breaks Prettier; update the
expression (where isHome is used to render the ahrefs meta tag in Root.tsx) to
remove the extra space before the closing brace so the conditional reads with no
trailing whitespace after "/>" (i.e., adjust the {isHome && <meta ... />}
rendering).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
dbde7c6 to
b42d5e7
Compare
b42d5e7 to
16ddd5e
Compare
Description
Add https://ahrefs.com/ domain verification
What problem is being solved?
How is it being solved?
What changes are made to solve it?
References
Review Checklist
mainSummary by CodeRabbit