Skip to content
This repository has been archived by the owner on Dec 5, 2022. It is now read-only.

Open Graph/RDFa + schema.org/JSON-LD structured data #6

Merged
merged 2 commits into from
Jan 5, 2015

Conversation

jeffposnick
Copy link
Contributor

R: @devnook & @ebidel

This PR contains both Open Graph general website data (specified using RDFa) and schema.org event data (specified using JSON-LD).

Providing Open Graph data via RDFa should be sufficient to get rich snippets for the main page when sharing using various social networks.

The schema.org markup follows the published guidelines for events, except it uses JSON-LD, which I believe is the new hotness.

Feedback about this approach is appreciated.

@jeffposnick jeffposnick changed the title Open Graph/RDFa + schema.org/JSON-LD structure data Open Graph/RDFa + schema.org/JSON-LD structured data Dec 2, 2014
"@type": "Place",
"name": "Todovenue",
"address": {
"@type" : "PostalAddress",
Copy link
Contributor

Choose a reason for hiding this comment

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

File a bug to remember to update these deets later.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Filed #7

@ebidel
Copy link
Contributor

ebidel commented Dec 2, 2014

LGTM.

@jeffposnick
Copy link
Contributor Author

I'll wait for @devnook to weigh in before merging.

@devnook
Copy link
Contributor

devnook commented Dec 2, 2014

@danbri Dan, can you chime in? What do you think of using jsonld vs microdata here? The project is about general structure for the next year's Google IO web app.

@jeffposnick
Copy link
Contributor Author

We got some external confirmation that JSON-LD should be fine, so I'm merging this PR.

jeffposnick added a commit that referenced this pull request Jan 5, 2015
Open Graph/RDFa + schema.org/JSON-LD structured data
@jeffposnick jeffposnick merged commit 7c88fb7 into master Jan 5, 2015
@jeffposnick jeffposnick deleted the structured-data branch January 5, 2015 16:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants