Skip to content
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

Frame hydra:statusCode. #4

Closed
wants to merge 1 commit into from
Closed

Conversation

warpr
Copy link
Contributor

@warpr warpr commented Nov 2, 2013

Frame hydra:statusCode, without it "[object Object]" is displayed on the "Invoke an Operation" modal.

... without it "[object Object]" is displayed on the "Invoke an Operation" modal.
@lanthaler
Copy link
Owner

This is a problem with framing at the moment. The result depends on whether the incoming data used a native integer or a typed literal. The current code works for native integers (which is what the HydraBundle e.g. emits). Your change would break that in favor of supporting typed literals. Thus, I won’t merge this PR for the time being. I think it should be handled directly by the JSON-LD processor. There’s already an issue for that at lanthaler/JsonLD#35

@warpr
Copy link
Contributor Author

warpr commented Nov 2, 2013

Ah, I didn't realize this would break it for the other case. I'm generating the JSON-LD document from turtle, I can probably change that conversion to output native types in the JSON-LD.

@lanthaler
Copy link
Owner

Yes, just set the useNativeTypes flag to true.

@warpr warpr closed this Jan 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants