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

Use og:type=website for collection #1294

Merged
merged 1 commit into from
Feb 18, 2022
Merged

Use og:type=website for collection #1294

merged 1 commit into from
Feb 18, 2022

Conversation

michenly
Copy link
Contributor

<meta property="og:type" content="product.group" /> is not actually a valid value. Collection page should default to <meta property="og:type" content="website" /> instead.

cc: @jackson-lo

Why are these changes introduced?

Fixes #0.

What approach did you take?

Other considerations

Demo links

Checklist

`<meta property="og:type" content="product.group" />` is not actually a valid value. Collection page should default to `<meta property="og:type" content="website" />` instead.

cc: @jackson-lo
@michenly michenly self-assigned this Jan 31, 2022
@michenly michenly requested a review from a team January 31, 2022 20:15
Copy link
Contributor

@ludoboludo ludoboludo left a comment

Choose a reason for hiding this comment

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

Thanks for bringing this up.
If this needs to be changed, I think we will have to follow up to get it updated here in our docs as well: https://shopify.github.io/liquid-code-examples/example/open-graph-tags

@michenly
Copy link
Contributor Author

michenly commented Feb 1, 2022

@ludoboludo would you be able to do that change?

@ludoboludo
Copy link
Contributor

@ludoboludo would you be able to do that change?

No I think I would need to find which team owns that. I'd start asking in #docs.

Copy link
Contributor

@ludoboludo ludoboludo left a comment

Choose a reason for hiding this comment

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

I'll reach out to docs 👍

@michenly
Copy link
Contributor Author

@ludoboludo is there a way you can get another reviewer for me or use your power to merge?

@andrewetchen andrewetchen self-requested a review February 18, 2022 16:44
Copy link
Contributor

@andrewetchen andrewetchen left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks @michenly

@michenly michenly merged commit e6bad9e into main Feb 18, 2022
@michenly michenly deleted the update-collection-seo branch February 18, 2022 18:07
phapsidesGT pushed a commit to Gravytrain-UK/gt-shopify-dawn-theme that referenced this pull request Sep 3, 2024
`<meta property="og:type" content="product.group" />` is not actually a valid value. Collection page should default to `<meta property="og:type" content="website" />` instead.

cc: @jackson-lo
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.

3 participants