Skip to content

fix: update metadata URLs and add Open Graph images for pages#143

Merged
chef0111 merged 1 commit intomainfrom
staging
Feb 2, 2026
Merged

fix: update metadata URLs and add Open Graph images for pages#143
chef0111 merged 1 commit intomainfrom
staging

Conversation

@chef0111
Copy link
Copy Markdown
Owner

@chef0111 chef0111 commented Feb 2, 2026

Summary by CodeRabbit

  • Chores
    • Enhanced social media preview display by adding image references to Open Graph and Twitter metadata across multiple pages.
    • Updated Twitter card type to "summary_large_image" for richer preview appearances.
    • Streamlined canonical URL paths for improved SEO consistency.

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dev4room Ready Ready Preview, Comment Feb 2, 2026 4:03pm

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 2, 2026

Walkthrough

Three dashboard page metadata configurations were updated to replace baseUrl-based dynamic URLs with static absolute paths and to add Open Graph and Twitter image references along with the summary_large_image card type across home, community, and tags pages.

Changes

Cohort / File(s) Summary
Dashboard metadata updates
src/app/(root)/(dashboard)/(home)/page.tsx, src/app/(root)/(dashboard)/community/page.tsx, src/app/(root)/(dashboard)/tags/page.tsx
Removed baseUrl import and replaced dynamic URL construction with static absolute paths in OpenGraph and canonical URLs. Added og:image and Twitter image references pointing to /images/og-image.png. Added Twitter card type summary_large_image to all three pages.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

🐰 Social cards bloom bright today,
With images that light the way,
Static paths replace the old,
Our metadata stories boldly told!
Hops of joy for every page! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes across all three files: updating metadata URLs from baseUrl-based to absolute paths and adding Open Graph/Twitter images.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch staging

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
Owner Author

@chef0111 chef0111 left a comment

Choose a reason for hiding this comment

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

Approve

Copy link
Copy Markdown

@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: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/app/(root)/(dashboard)/tags/page.tsx (1)

11-32: ⚠️ Potential issue | 🟠 Major

Add metadataBase configuration to the root layout.

The relative paths for openGraph.url and alternates.canonical (lines 19, 30) will not be resolved to absolute URLs without metadataBase configured in the root layout. This causes social media crawlers to receive relative paths that won't work correctly. Configure metadataBase in src/app/layout.tsx or src/app/layout.js.

The OG image at public/images/og-image.png exists and is correctly referenced.

@chef0111 chef0111 merged commit 724e72b into main Feb 2, 2026
5 of 6 checks passed
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.

1 participant