-
Notifications
You must be signed in to change notification settings - Fork 21
feat(rdf): allow front-end to get available SHACL validation sets #5225
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
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.
Code looks good, minor comment
private static void setYamlFormat(Context ctx) { | ||
// application/yaml does not show output in browser but downloads instead, so uses suffix: | ||
// https://www.iana.org/assignments/media-type-structured-suffix/media-type-structured-suffix.xhtml | ||
ctx.contentType("text/plain+yaml"); |
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.
Constant
@@ -988,6 +989,8 @@ public void testRdfApiRequest() { | |||
final String defaultContentType = "text/turtle"; | |||
final String jsonldContentType = "application/ld+json"; | |||
final String ttlContentType = "text/turtle"; | |||
final String defaultContentTypeWithCharset = "text/turtle; charset=utf-8"; // charset is ignored | |||
final String yamlContentType = "text/plain+yaml"; |
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.
Zodat je he hem hier ook niet dubbel hoeft te gebruiken.
|
What are the main changes you did
TODO:
How to test
Checklist