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

Remove additional itemprop/property attribute on 'Book' example #185

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@unor
Contributor

unor commented Nov 23, 2014

A Microdata/RDFa example had two itemprop/property elements on the same element, one of which contained an invalid property value (Fiction).

I guess it was intended to use content="Fiction" instead, but note that Microdata doesn’t allow this on span (see #184) and the JSON-LD example uses "fictional" instead of "Fiction", too.

Fixed 'Book' example
An example had two itemprop/property elements on the same element, one of which contained no valid property.

@unor unor changed the title from 'Book' example contained two itemprop/property attributes on same element to Remove additional itemprop/property attribute on 'Book' example Nov 23, 2014

@danbri danbri modified the milestones: 2015 Q1, sdo-stantz release Jan 22, 2015

@danbri danbri self-assigned this Jan 22, 2015

@danbri danbri referenced this pull request Jan 22, 2015

Merged

Fixed 'Book' example #272

@danbri

This comment has been minimized.

Show comment
Hide comment
@danbri

danbri Jan 22, 2015

Contributor

Thanks! Merged in via #272 (I pulled it into sdo-stantz, our upcoming release, rather than master branch).

Contributor

danbri commented Jan 22, 2015

Thanks! Merged in via #272 (I pulled it into sdo-stantz, our upcoming release, rather than master branch).

@danbri danbri closed this Jan 22, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment