Add json-ld export#809
Conversation
Test Results 57 files + 1 57 suites +1 2m 34s ⏱️ +8s Results for commit 8e5045e. ± Comparison against base commit 954a2e3. This pull request removes 3 and adds 122 tests. Note that renamed tests count towards both. |
| } | ||
|
|
||
| @Test | ||
| void shouldEmitProfileLinkForJsonLd() { |
There was a problem hiding this comment.
if you omit profile in request, it will default to our verbose json?
There was a problem hiding this comment.
This concern existed before this change, but I have now added a test that explicitly pins the default representation to JSON-UTF-8
| public static final String STATUS = "status"; | ||
| public static final String SUFFIX = ")"; | ||
| public static final String TAGS = "tags"; | ||
| public static final String APPLICATION_LD_JSON = "application/ld+json"; |
There was a problem hiding this comment.
have a look at static String[] forMediaType(MediaType mediaType, Collection<String> defaultIncludedFields)
Could you add the needed json fields here to limit "Entity Too Large" issues?
There was a problem hiding this comment.
(this comment is more or less randomly placed btw)
There was a problem hiding this comment.
maybe we should restructure the pattern here so that included fields are enforced next time we add a new profile/format
No description provided.