Add a wrapping @context tag to JSON-LD generator
#454
Conversation
mristin
changed the title
@context tag to JSON-LD generator
|
@VladimirAlexiev and/or @ethieblin Could you please explain a bit the reason behind this change so that I can write a proper description in the pull request? It doesn't have to be long, but just something that allows the future readers of the commit history to understand the background. |
|
@ethieblin please also see the failing CI -- I think you need to re-format the code (by calling |
This issue is related to admin-shell-io/aas-specs#386 It is needed so that the context can be served directly at a given URL and digested in a JSON-LD document
ethieblin
force-pushed
the
jsonld-context
branch
from
c913ce4 to
a2c82cb
Compare
|
@mristin Is it ok for you now ? |
|
Thanks, @ethieblin, LGTM! |
|
@mristin According to the JSON-LD spec, a remote context must be a valid JSON-LD document, which means that it needs to include the word |
|
Thanks, @VladimirAlexiev ! I already merged in the pull request, but the readers can look up this discussion for further info. |
This PR is related to admin-shell-io/aas-specs#386.
It adds a wrapping "@context" tag around the JSON-LD context.
It is needed so that the context can be served directly at a given URL and digested in a JSON-LD document.