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

chore: Updates website structured data #2662

Merged
merged 3 commits into from
Feb 3, 2025
Merged

Conversation

hellofanny
Copy link
Contributor

@hellofanny hellofanny commented Jan 31, 2025

What's the purpose of this pull request?

Adds name and publisher properties to website schema

How it works?

Allow adding the following WebSite schema proprieties:

  • name: website name.
  • publisher: the publisher of the creative work. (We are allowing a publisherId @id to identifier reference to the publisher. E.g.: #website)

Users can add data either through the headless CMS interface or by editing the discovery.config.js file.

Note: Data from the headless CMS takes precedence over data from discovery.config.js.

image

How to test it?

  1. Go to preview link, inspect and look for WebSite
image
  1. Copy the generated script and paste here to check if is a valid schema:
image

OR

  1. Test in storeframework account, go to CMS admin, changes the fields values -> Save and Publish.
  2. Wait for the new preview link and do the same process mention above (look for WebSite and validate the schema)

Starters Deploy Preview

vtex-sites/starter.store#677

References

https://developers.google.com/search/blog/2024/10/sitelinks-search-box
https://developers.google.com/search/docs/appearance/site-names#website

Copy link

vercel bot commented Jan 31, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
faststore-site ⬜️ Ignored (Inspect) Visit Preview Feb 3, 2025 4:58pm

Copy link

codesandbox-ci bot commented Jan 31, 2025

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@hellofanny hellofanny changed the title chore: Updates website structured data chore: Updates website structured data Jan 31, 2025
@hellofanny hellofanny marked this pull request as ready for review February 3, 2025 14:40
@hellofanny hellofanny requested a review from a team as a code owner February 3, 2025 14:40
@hellofanny hellofanny requested review from lariciamota and pedromtec and removed request for a team February 3, 2025 14:40
Copy link
Member

@eduardoformiga eduardoformiga left a comment

Choose a reason for hiding this comment

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

LGTM!

🎗️ updates vtex account in hCMS.

packages/core/src/pages/index.tsx Show resolved Hide resolved
@hellofanny hellofanny self-assigned this Feb 3, 2025
@hellofanny hellofanny merged commit 08efd74 into main Feb 3, 2025
8 checks passed
@hellofanny hellofanny deleted the chore/seo-website-update branch February 3, 2025 19:38
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