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
Use json-ld v0.6 #342
Use json-ld v0.6 #342
Conversation
If I may clarify, you should not have to use |
Great, thanks @timothee-haudebourg! |
I've added support for |
Excellent. I just remembered though about using |
Build with nightly fails because of missing implementation of This is using serde_json 1.0.74, generic-json v0.5.1 with |
Attempted to convert from
|
Conversion from IValue is implemented; the crate compiles and runs now. But RDF Deserialization is not working now. These changes could be examined to see if any obvious bugs have been introduced. Probably we should just use the RDF Serialization implementation being added in timothee-haudebourg/json-ld#29; then the implementation of it in ssi (which this PR currently changes to use Value instead of JsonValue) could be dropped. Also there is something going on in did-tezos with StdError: error in did-tezos
errors in toRdf
|
The more I think about it, the more I think I made a mistake here. I remember making some change specifically to acomodate the lack of
The |
Should be fixed in the latest version 0.6.0. I'm now using |
Attempting to integrate json-ld v0.6, using serde_json::Value again. |
Ensure generic-json serde_json impl
Issue with Other things mentioned in #342 (comment) still occur:
ssi crate test failures
|
Document JsonLdLoc error
Tested with timothee-haudebourg/json-ld#29.
Error in Error in did-tzhttps://github.com/spruceid/ssi/runs/5408065687?check_suite_focus=true#step:5:951
|
Will open a new PR for v0.8 |
json-ld
crate version0.5 (changelog)v0.6 is published to crates.io. To use new features ofjson-ld
, such as warnings (timothee-haudebourg/json-ld#26) or flattening (timothee-haudebourg/json-ld#29 in development),ssi
will need to be updated to usejson-ld
's new JSON interface, including in the JSON-LD algorithms implemented (Node Map Generation, and RDF Deserialization). The use ofJsonValue
(json
crate) is removed injson-ld
v0.5. Instead, generic-json is used. Injson-ld
v0.6,serde_json::Value
can be used without conversion (#342 (comment)).JsonValue
withserde_json::Value
Change fromNot needed: Use json-ld v0.6 #342 (comment), Use json-ld v0.6 #342 (comment)serde_json::Value
togeneric_json::Value
where needed.generic_associated_types
/ Rust Stable vs. Nightly