Skip to content

fix: update Open Graph metadata to use base URL and structured image data#146

Merged
chef0111 merged 2 commits intomainfrom
staging
Feb 3, 2026
Merged

fix: update Open Graph metadata to use base URL and structured image data#146
chef0111 merged 2 commits intomainfrom
staging

Conversation

@chef0111
Copy link
Copy Markdown
Owner

@chef0111 chef0111 commented Feb 3, 2026

Summary by CodeRabbit

  • Improvements
    • Enhanced metadata handling to improve how pages appear when shared on social media platforms with standardized URLs and optimized image definitions.

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 3, 2026

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

Project Deployment Actions Updated (UTC)
dev4room Ready Ready Preview, Comment Feb 3, 2026 3:07am

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 3, 2026

Walkthrough

Four dashboard page metadata configurations are updated to use a consistent baseUrl and enhanced image objects containing URL, width, height, and alt properties instead of simple string paths for Open Graph and Twitter social media sharing cards.

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
Imported baseUrl from constants and refactored metadata image definitions from string paths to objects containing url, width, height, and alt properties. Updated openGraph and twitter URLs to use baseUrl prefix for consistency.
Dynamic Question Page Metadata
src/app/(root)/(dashboard)/questions/[id]/page.tsx
Converted generateMetadata image arrays from simple string paths to structured objects with url, width, height, and alt fields for openGraph and twitter image metadata.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

🐰 The rabbit hops through metadata with glee,
BaseURLs strutting—consistency!
Image objects bloom with width and height,
Social cards now shimmer, polished and bright. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 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 modified files: updating Open Graph metadata to use baseUrl consistently and converting image definitions from strings to structured objects with metadata (url, width, height, alt).

✏️ 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

@chef0111 chef0111 merged commit f322bb2 into main Feb 3, 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