-
Notifications
You must be signed in to change notification settings - Fork 71
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
Can we output expanded JSON-LD by default? #422
Comments
Talking to @acoburn has revealed the use of "profile" within the accept header. Fedora 4 is doing something like Now, we don't strictly enforce Accept headers, but we could still look for this and act on it. 👍 for a standard way of describing which jsonld operation you want to perform upon request. |
The JSON-LD spec describes this here: https://www.w3.org/TR/json-ld/#application-ld-json |
OK, can do that. Have to see where do i have to cut/break/hack to be able to have a clean header access. If not, can make it the default via config options. Is that right for you @dannylamb? |
@DiegoPino right now all i need is the serializer to output expanded jsonld. I'm outside of the HTTP layer and I'm using it to publish to a queue. That will unblock me for the time being. We can map out the specifics for headers and GET requests separately as part of #398. |
@dannylamb will do that for today. 😄 |
Hey @DiegoPino, can you give this a look when you have a chance? https://github.com/DiegoPino/claw-jsonld/pull/1 |
@dannylamb yes, i'm my radar. Just finished fedora4camp and moved to my new apartment. You will get news on this on Friday. |
@DiegoPino Those are definitely more important things than this. Just whenever you get a chance, man. |
@DiegoPino what do you think? I want to use the jsonld serializer to return responses AND put jsonld on the queue. I'd prefer it be expanded when getting passed to Fedora. Can we make that the default?
I'd like to explore other options (like auto-generating contexts and allowing the user to request different representations through the use of headers) later, but for now would like to simplify things if possible.
The text was updated successfully, but these errors were encountered: