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.

Sign up for GitHub

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

Jump to bottom

Second context needed in VC Documentation? #60

Closed
simibac opened this issue Feb 17, 2021 · 3 comments · May be fixed by #80
Closed

Second context needed in VC Documentation? #60

simibac opened this issue Feb 17, 2021 · 3 comments · May be fixed by #80

Comments

@simibac
Copy link

simibac commented Feb 17, 2021

Shouldn't there be a second @context added in the example VC in the docs such that the VC can be serialised? Otherwise, how should the verifier know what kind of community rank is about?

https://github.com/uport-project/veramo-website/blame/master/docs/fundamentals/verifiable_data.md#L28
@simibac simibac changed the title Second context needed in VC Documentation Second context needed in VC Documentation? Feb 17, 2021
@mirceanis
Copy link
Contributor

Yes. You are right.
The way we used @context go far has not been very friendly to JSON-LD processors.
This will certainly change soon since we're adding Ld-proof support.
For this example there is no ready-made context that I know of.

@rado0x54
Copy link

Updated link to that issue: https://github.com/uport-project/veramo-website/blob/main/docs/basics/verifiable_data.md

@rado0x54 rado0x54 mentioned this issue Jun 28, 2021
Open
@mirceanis
Copy link
Contributor

Closing this as outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update VC example to real VC with LD Signature uport-project/veramo-website
3 participants
@mirceanis @rado0x54 @simibac