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

Update the 'image' property description #103

Merged
merged 1 commit into from Sep 10, 2014
Merged

Conversation

@dbs
Copy link
Contributor

@dbs dbs commented Aug 20, 2014

We now describe that the image property has a range of both URL and
ImageObject, allowing for more flexible usage of the property.

Fixes #99.

Signed-off-by: Dan Scott dan@coffeecode.net

@@ -6473,7 +6473,7 @@ Note that Event uses startDate/endDate instead of startTime/endTime, even when d
</div>
<div typeof="rdf:Property" resource="http://schema.org/image">
<span class="h" property="rdfs:label">image</span>
<span property="rdfs:comment">URL of an image of the item.</span>
<span property="rdfs:comment">An image of the item. This can a &lt;a href=&quot;http://schema.org/URL&quot;&gt;URL&lt;/a&gt; or a fully described &lt;a href=&quot;http://schema.org/ImageObject&quot;&gt;ImageObject&lt;/a&gt;.</span>

This comment has been minimized.

@scor

scor Aug 20, 2014
Contributor

This can be a.

We now describe that the image property has a range of both URL and
ImageObject, allowing for more flexible usage of the property.

Fixes #99.

Signed-off-by: Dan Scott <dan@coffeecode.net>
@dbs dbs force-pushed the dbs:image_description branch from 4ea1adc to 90c8085 Aug 20, 2014
@dbs
Copy link
Contributor Author

@dbs dbs commented Aug 20, 2014

Right you are, @scor -- thanks for the missing verb! Updated.

@danbri
Copy link
Contributor

@danbri danbri commented Sep 3, 2014

Thanks. Did you mean that it fixes #100 ?

(#100 not #99)

danbri added a commit that referenced this pull request Sep 10, 2014
Update the 'image' property description
@danbri danbri merged commit 8cb080a into schemaorg:master Sep 10, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants