Skip to content

Enhance blog post schema markup with comprehensive Article structured data#3506

Open
sumitshinde-84 wants to merge 2 commits intomainfrom
improve-article-schema
Open

Enhance blog post schema markup with comprehensive Article structured data#3506
sumitshinde-84 wants to merge 2 commits intomainfrom
improve-article-schema

Conversation

@sumitshinde-84
Copy link
Collaborator

Description

Related Issue(s)

Checklist

  • I have read the contribution guidelines
  • I have considered the performance impact of these changes
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated

@ZJvandeWeg ZJvandeWeg requested a review from Copilot July 18, 2025 08:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the blog post template with comprehensive structured data markup by adding Article schema.org JSON-LD to improve SEO and search engine understanding of blog content.

  • Adds complete Article schema markup with metadata fields including headline, description, dates, and language
  • Implements dynamic author information with Person schema including social media profiles and organizational affiliation
  • Includes conditional FAQ schema markup and blog relationship metadata for enhanced content structure

Comment on lines +46 to +48
{% if people[author].linkedin %}
"sameAs": ["https://www.linkedin.com/in/{{ people[author].linkedin }}"{% if people[author].github %}, "https://github.com/{{ people[author].github }}"{% endif %}],
{% endif %}
Copy link

Copilot AI Jul 18, 2025

Choose a reason for hiding this comment

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

The sameAs array is only created when linkedin exists, but should include all available social profiles. Consider restructuring to build the array more comprehensively and handle cases where only github exists without linkedin.

Suggested change
{% if people[author].linkedin %}
"sameAs": ["https://www.linkedin.com/in/{{ people[author].linkedin }}"{% if people[author].github %}, "https://github.com/{{ people[author].github }}"{% endif %}],
{% endif %}
{% set sameAs = [] %}
{% if people[author].linkedin %}
{% set sameAs = sameAs.concat(["https://www.linkedin.com/in/{{ people[author].linkedin }}"]) %}
{% endif %}
{% if people[author].github %}
{% set sameAs = sameAs.concat(["https://github.com/{{ people[author].github }}"]) %}
{% endif %}
{% if sameAs.length > 0 %}
"sameAs": {{ sameAs | dump | safe }},
{% endif %}

Copilot uses AI. Check for mistakes.
Comment on lines +70 to +71
"width": 396,
"height": 215
Copy link

Copilot AI Jul 18, 2025

Choose a reason for hiding this comment

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

The logo dimensions (396x215) suggest an unusual aspect ratio. Verify these dimensions are correct for the referenced logo file, as incorrect dimensions can affect schema markup validation.

Suggested change
"width": 396,
"height": 215
"width": 512,
"height": 512

Copilot uses AI. Check for mistakes.
"url": "{{ page.url | url | toAbsoluteUrl }}",
"inLanguage": "en-US",
{% if tags %}
{% set filtered_tags = tags | reject('equalto', 'posts') | reject('equalto', 'node-red') | list %}
Copy link

Copilot AI Jul 18, 2025

Choose a reason for hiding this comment

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

Hardcoded tag filtering ('posts', 'node-red') should be configurable or documented. Consider moving these exclusions to a configuration file or adding a comment explaining why these specific tags are filtered.

Suggested change
{% set filtered_tags = tags | reject('equalto', 'posts') | reject('equalto', 'node-red') | list %}
{% set excludedTags = ['posts', 'node-red'] %} {# Tags to exclude from the keywords list. Update as needed. #}
{% set filtered_tags = tags | reject('in', excludedTags) | list %}

Copilot uses AI. Check for mistakes.
@sumitshinde-84
Copy link
Collaborator Author

@ZJvandeWeg This PR is not ready for review !

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.

2 participants