Skip to content

Remove invalid reviewedBy property from JSON-LD schema#16926

Closed
minimalsm wants to merge 1 commit into
devfrom
fix/remove-invalid-reviewedby-schema
Closed

Remove invalid reviewedBy property from JSON-LD schema#16926
minimalsm wants to merge 1 commit into
devfrom
fix/remove-invalid-reviewedby-schema

Conversation

@minimalsm
Copy link
Copy Markdown
Contributor

@minimalsm minimalsm commented Dec 17, 2025

Summary

  • Remove reviewedBy property from all JSON-LD structured data
  • Fixes AHREFS warning: "reviewedBy - Unexpected property for Article"

The reviewedBy property is not valid for Article or WebPage schema types according to schema.org. This was causing structured data validation warnings.

Files changed: 40 page-jsonld.tsx files across the app

Test plan

  • Verify pages still render correctly
  • Test structured data with Google Rich Results Test
  • Confirm no new schema warnings in AHREFS

The `reviewedBy` property is not valid for Article or WebPage schema types
according to schema.org. This was flagged by AHREFS site audit.

Removes reviewedBy from all 40 page-jsonld.tsx files.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 17, 2025

Deploy Preview for ethereumorg failed.

Name Link
🔨 Latest commit cbf9c4c
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/69431f5f0b3fec00080f5605

@wackerow
Copy link
Copy Markdown
Member

wackerow commented Jan 6, 2026

Claude review:


This PR is no longer needed.

Here's the situation:

Summary

PR #16926 attempts to remove the invalid reviewedBy property from JSON-LD structured data across 40 files. However, dev branch has already undergone a complete JSON-LD refactoring (merged via PR #16801 "data layer") that:

  1. Replaced the old JSON-LD structure with a new @graph-based architecture
  2. Introduced shared organization references from @/lib/utils/jsonld
  3. Already removed reviewedBy as part of that refactoring

Rebase Results

When attempting to rebase PR #16926 onto the current dev branch, all 40 files had merge conflicts because the code the PR was trying to modify no longer exists in its original form.

Recommendation

Close this PR without merging. The issue it was trying to fix (AHREFS warning about invalid reviewedBy property) has already been resolved by the data layer refactoring.


Structured Review

Verdict: CLOSE (No longer needed)

Aspect Status
Code quality N/A - PR is obsolete
Original fix correctness ✅ The fix was valid for the old codebase
Current relevance ❌ Changes already superseded by PR #16801
Merge conflicts ❌ 40/40 files conflict

@wackerow wackerow closed this Jan 6, 2026
@github-actions github-actions Bot added the abandoned This has been abandoned or will not be implemented label Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

abandoned This has been abandoned or will not be implemented

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants