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

SEO changes #506

Merged
merged 3 commits into from
Aug 27, 2021
Merged

SEO changes #506

merged 3 commits into from
Aug 27, 2021

Conversation

ludoboludo
Copy link
Contributor

Why are these changes introduced?

Fixes #498

What approach did you take?

Added some logic to the <title> to follow the recommendations we give on our help docs while this other issue is still open/discussed.

Also added the <script type="application/ld+json"> where necessary.

Also updated the link to the Dawn theme for theme_documentation_url

Other considerations

I still need to test it out with Google's tool but since we keep a password on the theme I can't.

Demo links

Checklist

@tauthomas01 tauthomas01 self-assigned this Aug 27, 2021
Copy link
Contributor

@tauthomas01 tauthomas01 left a comment

Choose a reason for hiding this comment

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

Works well

Featured Product
alt

Article
alt

Product page
alt

Blogs pagination
image

For tags, do you know where can we test?

@@ -4,7 +4,7 @@
"theme_name": "Dawn",
"theme_version": "1.1.0",
"theme_author": "Shopify",
"theme_documentation_url": "https://shopify.dev/themes/tools/dawn",
"theme_documentation_url": "https://help.shopify.com/manual/online-store/themes/os20/themes-by-shopify/dawn",
Copy link
Contributor

Choose a reason for hiding this comment

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

The link does not seem to work. Is it expected that it will be deployed in the future?

Copy link
Contributor Author

@ludoboludo ludoboludo Aug 27, 2021

Choose a reason for hiding this comment

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

Yes! Sorry I should have added context for this. It's going to made available when the theme is launched.

As for the tags, it's usually needed when filtering used with tags assigned in the admin. We do filtering a different way now but it might still be useful in case anything else leverages it ? (apps, etc). You can add tags to products and also blog articles.
current_tags object

And right now to answer your question, I don't think we have a way to test it based on our theme and demo store setup.

Copy link
Contributor

@tauthomas01 tauthomas01 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.

Tested tags by hardcoding the URL (e.g. https://os2-demo.myshopify.com/collections/all/handbag)
image

@ludoboludo ludoboludo merged commit 767af05 into main Aug 27, 2021
@ludoboludo ludoboludo deleted the seo-changes branch August 27, 2021 20:41
@gregjotau
Copy link

I see JSON LD is added for the:

Articles
Products
Store in general

I think that should be sufficient and covers most that other apps offer today, are there any plans to add additional schemas or build upon the one added? (not sure if it uses all possible fields etc. in the specs that might be useful)

image

I merged the change and will test with a schema tester for famme.no

@gregjotau
Copy link

https://search.google.com/test/rich-results?id=_oY0RmI8j34peNJQ8ujxUA

Missing a few fields that might be useful? I must admit I do not 100% know the need for those :) It might depend from brand to brand also I guess.

image

phapsidesGT pushed a commit to Gravytrain-UK/gt-shopify-dawn-theme that referenced this pull request Sep 3, 2024
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.

SEO Requirements
4 participants