-
-
Notifications
You must be signed in to change notification settings - Fork 387
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
Feature/dataset jsonld comp #162
Feature/dataset jsonld comp #162
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR, just one small change needed.
"@type": "Dataset", | ||
"description": "${description}", | ||
"name": "${name}"${ | ||
license |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At the moment if license
is added it will be invalid JSON due to no trailing comma on name
.
To fix this if you put name
last and then update license to always have trailing comma like so: "license": "${license}",
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi!
I will actually be a comma on name if license is provided.
See line 23.
The result will be like this:
{
"@context": "http://schema.org",
"@type": "Dataset",
"description": "The description needs to be at least 50 characters long",
"name": "name of the dataset",
"license": "https//www.example.com"
}
The solution is the same as providerUrl in course component.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😅 Whoops missed that. Perfect 👌
"@type": "Dataset", | ||
"description": "${description}", | ||
"name": "${name}"${ | ||
license |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😅 Whoops missed that. Perfect 👌
Added jsonld for dataset.