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

Discovery: Stories Ranking Well in Search Results #3179

Closed
MackenzieHartung opened this issue Sep 4, 2019 · 8 comments
Closed

Discovery: Stories Ranking Well in Search Results #3179

MackenzieHartung opened this issue Sep 4, 2019 · 8 comments
Assignees
Milestone

Comments

@MackenzieHartung
Copy link

To do: What requirements are needed for Stories to rank well in search results? If jetpack and yoast do not cover the requirements identify other plugins we need to engage with to ensure we provide users the proper capabilities.

@spacedmonkey
Copy link
Contributor

Some work with Yoast seems to already been done here. See Yoast/wordpress-seo#13446

@spacedmonkey
Copy link
Contributor

This is now been merged. Automattic/jetpack#13416

@spacedmonkey
Copy link
Contributor

Recommendation:

There seems to be have been a fair bit of work already done on this. There are two PR already open for Yoast and Jetpack already. See
Yoast/wordpress-seo#13446
Automattic/jetpack#13416

The yoast PR still needs a little work, so that is an action. Otherwise, the AMP template seem pretty SEO friendly already with work being put into ALT tags and screen reader text going a long way to help with the complete visual elements on the stories.

The only one item I might flag, is by default many of the stories do no contain any h tags, speically H1s. This is possible by adding a text block and changing the text type. However, this option is a somewhat hidden and the is no visual indicator when the text type is changed. I believe it might be better to have another header block and break out and remove option from the text block. This might push users to add header tags to they pages and improve SEO.

@spacedmonkey
Copy link
Contributor

@westonruter I noticed that the two PRs for Yoast and Jetpack, don't override amp_get_schemaorg_metadata. If either of these plugins are active, should the open graph values generated by these plugins override the defaults in the AMP plugin?

@westonruter
Copy link
Member

@spacedmonkey In the case of Yoast, the AMP-generated Schema.org metadata is overridden entirely. In the case of Jetpack, they do integrate with the AMP-generated Schema.org meta here: https://github.com/Automattic/jetpack/blob/a9959f247ae98b07aeb5a3c1814477c30a0744ec/3rd-party/class.jetpack-amp-support.php#L39

I think there should be a merge, not an outright override on either end.

@spacedmonkey
Copy link
Contributor

As a proof of concept, I added a commit Yoast/wordpress-seo@ba929ab to the existing yoast PR. This overrides article schema and replaces it with schema generated by the amp plugin.

@MackenzieHartung As the action point to this ticket, should I create a ticket to track the work on the yoast plugin on our board?

@spacedmonkey
Copy link
Contributor

Jetpack 7.8 beta 1 released - https://github.com/Automattic/jetpack/releases/tag/7.8-beta
This contains SEO fix for AMP Stories.

@spacedmonkey
Copy link
Contributor

It is clear what the actions are for this.

  1. Integration Yoast ld data with AMP stories data.
  2. Integration Jetpack ld data with AMP stories data.

Both of these actions are work that is required on these plugins and not the AMP plugin. I will create issues, but the PRs will not be here.

Worth noting that the Jetpack work is already complete and will be released in 7.8 of the plugin. For that reason I will not create a issue to track it.

@swissspidy swissspidy added this to the v1.3.1 milestone Oct 2, 2019
@kmyram kmyram modified the milestones: v1.3.1, v1.4 Oct 8, 2019
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

No branches or pull requests

5 participants