Skip to content
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.

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

Add 9 missing supported Post Types #1915

Merged
merged 1 commit into from
Oct 12, 2023

Conversation

vaurdan
Copy link
Contributor

@vaurdan vaurdan commented Oct 11, 2023

Description

This pull request, related to issue #1899, addresses the bug where WP Parse.ly supports only 8 out of 17 @types for post content in the JSON-LD output. To fix this issue, this PR adds support for the missing @types to ensure that the WordPress plugin provides complete and accurate JSON-LD data representation.

Motivation and context

This change is necessary to align the plugin with Parse.ly's post @types and to prevent incomplete data representation.

How has this been tested?

I have tested this change by implementing and using the provided filter in #1899 for a couple of the newly supported @types. I have validated that, without this change, when using repeated_metas, the post type metadata would return index. With the new types added, it now returns post.

@vaurdan vaurdan added this to the 3.11.0 milestone Oct 11, 2023
@vaurdan vaurdan requested a review from a team as a code owner October 11, 2023 15:38
@vaurdan vaurdan linked an issue Oct 11, 2023 that may be closed by this pull request
Copy link
Collaborator

@acicovic acicovic left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks! 🙂

@vaurdan vaurdan merged commit 290d5ec into develop Oct 12, 2023
36 checks passed
@vaurdan vaurdan deleted the add/additional-supported-post-types branch October 12, 2023 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Additional Supported Post Schema @types
2 participants