Enhancement: Add JSON-LD structured data to the home page (#117)#139
Enhancement: Add JSON-LD structured data to the home page (#117)#139coolss21 wants to merge 2 commits into
Conversation
|
@rhoggs-bot-test-account is attempting to deploy a commit to the Rishi Bhardwaj's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Warning Review limit reached
More reviews will be available in 3 minutes and 40 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe PR adds JSON-LD structured data to the home page by defining a jsonLd constant with WebSite schema (name, description, URL, context) and injecting it as an ChangesJSON-LD Structured Data
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
🚥 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)
Comment |
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 `@project/src/app/page.tsx`:
- Around line 178-195: The jsonLd script is being injected client-side in
page.tsx (and thus lost to SSR) — move the structured data into a
server-rendered location: either create a WebsiteJsonLd component modeled on the
existing RoadmapJsonLd and import/use it from the root RootLayout (server
layout) or add the jsonLd script directly into RootLayout so it renders inside
<head> during SSR; remove the client-side script block (jsonLd injection) from
page.tsx and ensure no "use client" forces the JSON-LD to render only on the
client.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
Resolves #117
Overview
This PR adds
WebSiteSchema.org JSON-LD to the rootpage.tsxcomponent.Why this helps
Providing structured data allows search engines to better understand the project's purpose and context, improving the site's search visibility and SEO footprint.
Changes included
<script type="application/ld+json">tag within thepage.tsxfile containing the site name, URL, and project description.Summary by CodeRabbit