Skip to content

chore: add ahrefs verification#1276

Merged
rhamzeh merged 1 commit into
mainfrom
chore/add-ahrefs-verification
Jun 4, 2026
Merged

chore: add ahrefs verification#1276
rhamzeh merged 1 commit into
mainfrom
chore/add-ahrefs-verification

Conversation

@rhamzeh
Copy link
Copy Markdown
Member

@rhamzeh rhamzeh commented Jun 4, 2026

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

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

Summary by CodeRabbit

  • Chores
    • Added a verification meta tag to the home page for enhanced site configuration management.

@rhamzeh rhamzeh requested review from aaguiarz and Copilot June 4, 2026 14:19
@rhamzeh rhamzeh requested review from a team as code owners June 4, 2026 14:19
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jun 4, 2026

Review Change Stack

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bd676345-2714-40a9-9683-3dac981bc5c5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

This 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 isHome route check.

Changes

Home page Ahrefs verification

Layer / File(s) Summary
Home page verification meta tag
src/theme/Root.tsx
The Root component adds an ahrefs-site-verification meta tag with a hardcoded verification value to the document <Head> when the current route is the home page.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: add ahrefs verification' directly and clearly summarizes the main change: adding Ahrefs domain verification to the site.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/add-ahrefs-verification

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

Copilot AI left a 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 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.

Comment thread src/theme/Root.tsx Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 4, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-06-04 14:30 UTC

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f5741234-7683-4258-8c45-672b285d18cc

📥 Commits

Reviewing files that changed from the base of the PR and between 2d0d4f4 and dbde7c6.

📒 Files selected for processing (1)
  • src/theme/Root.tsx

Comment thread src/theme/Root.tsx Outdated
@rhamzeh rhamzeh force-pushed the chore/add-ahrefs-verification branch from dbde7c6 to b42d5e7 Compare June 4, 2026 14:24
@rhamzeh rhamzeh enabled auto-merge (squash) June 4, 2026 14:24
@rhamzeh rhamzeh force-pushed the chore/add-ahrefs-verification branch from b42d5e7 to 16ddd5e Compare June 4, 2026 14:26
@rhamzeh rhamzeh merged commit 05e46a6 into main Jun 4, 2026
14 checks passed
@rhamzeh rhamzeh deleted the chore/add-ahrefs-verification branch June 4, 2026 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants