Skip to content

Add og:url meta tag#117

Merged
hasparus merged 1 commit into
mainfrom
add-og-url
Jul 6, 2026
Merged

Add og:url meta tag#117
hasparus merged 1 commit into
mainfrom
add-og-url

Conversation

@hasparus

@hasparus hasparus commented Jul 6, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Owner

Blog posts (and all pages) were missing og:url. Add it in BaseLayout reusing the existing canonical URL, so it stays in sync with the canonical link.

Summary by CodeRabbit

  • New Features
    • Added an Open Graph URL tag to improve how shared pages appear in link previews and social platforms.

@coderabbitai

coderabbitai Bot commented Jul 6, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: dee75a12-ca4e-4119-86c7-edc8b5e11481

📥 Commits

Reviewing files that changed from the base of the PR and between 129e9e1 and fe059d4.

📒 Files selected for processing (1)
  • src/layouts/BaseLayout.astro
📝 Walkthrough

Walkthrough

A single meta tag for og:url was added to the <head> of BaseLayout.astro, using the existing computed canonical URL value. No other changes were made.

Changes

Open Graph URL meta tag

Layer / File(s) Summary
Add og:url meta tag
src/layouts/BaseLayout.astro		 Added a <meta property="og:url"> tag in the head, populated with the previously computed canonical URL.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

A single tag, a single line,
og:url now shares the shine,
Canonical hops right into place,
One rabbit-hop, a tidy trace. 🐇

🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding an og:url meta tag.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Accessible E2e Tests ✅ Passed PR only changes BaseLayout.astro; no e2e tests were added or modified, so this accessibility selector check is not applicable here.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch add-og-url

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.

@hasparus hasparus merged commit 225e9bf into main Jul 6, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@hasparus