Skip to content

Fix json-ld #208

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 3, 2025
Merged

Fix json-ld #208

merged 1 commit into from
Sep 3, 2025

Conversation

svekars
Copy link
Contributor

@svekars svekars commented Sep 3, 2025

  • Change Article to TechArticle for better SEO
  • Removed articleBody as it causes error in structure due to LaTeX formatting.
  • Updated the "headline" field to correctly capture headline. Previous implementation caused invalid JSON when page titles contained special characters or code syntax
  • Added name field which is required for this type of schema.

@meta-cla meta-cla bot added the cla signed label Sep 3, 2025
@svekars svekars requested a review from sekyondaMeta September 3, 2025 18:17
@svekars svekars merged commit 588d62e into pytorch_sphinx_theme2 Sep 3, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sekyondaMeta sekyondaMeta sekyondaMeta approved these changes

Assignees

No one assigned

Labels

cla signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@svekars @sekyondaMeta