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

comment and comment_plain field always empty for datatypes and types #58

Open
udiedrichsen opened this issue Jul 2, 2014 · 2 comments

Comments

@udiedrichsen
Copy link

I am using the JSON data provided by
http://schema.rdfs.org/all.json

but for all datatypes and types are the fields comments and commtents_plain empty:
datatypes": {
"Boolean": {
"ancestors": [
"DataType"
],
"comment": "",
"comment_plain": "",
"id": "Boolean",
"instances": [
"False",
"True"
],
"label": "Boolean",
"properties": [],
"specific_properties": [],
"subtypes": [],
"supertypes": [
"DataType"
],
"url": "http://schema.org/Boolean"
},

and

"WearAction": {
"ancestors": [
"Thing",
"Action",
"ConsumeAction",
"UseAction"
],
"comment": "",
"comment_plain": "",

Would be nice, if the comments could be filled. For all properties the comment fields are ok.

Cheers,

Uli

whizkid77 added a commit to whizkid77/schema-org-rdf that referenced this issue Oct 17, 2014
Fix parsing where comment and comment_plain are empty
@redaktor
Copy link

👍
@udiedrichsen @whizkid77 – just noticed that as well while working on a node.js module.
Would be nice if this could be fixed in the 'scraper'.
Meanwhile:
I compared the results of .types with our node.js JSON and put the missing .types-comments in this gist!

You can ignore the links attribute - it contains links within the comments.
Why? [would be also a "feature request"/"enhancement" for this project]
I think there is a JSON specification for specifying schema.org - and that is JSON Schemanice read here
For example when the comment is saying "use ISO 8601 duration format" this can be specified in JSON Schema. A discussion is here.

@Jonadabe
Copy link

Jonadabe commented Jan 5, 2015

👍

cygri added a commit that referenced this issue Mar 26, 2015
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

No branches or pull requests

2 participants