-
Notifications
You must be signed in to change notification settings - Fork 28
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
feat: add did contexts #712
Conversation
babed59
to
227ac42
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
* See https://w3c-ccg.github.io/security-vocab/. | ||
* This document is extended by the context file available under the [[KILT_DID_CONTEXT_URL]]. | ||
*/ | ||
export const W3C_SECURITY_CONTEXT_URI = 'https://w3id.org/security/v2' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is also a URL tho, right? What about renaming them all to URI? Since the IPFS link is definitely not a URL.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually I think I wanted to name it "URL" bc last thing I heard was that the distinction IRI / URI / URL is being abandoned in favour of calling everything "URL". Why is an ipfs link not a URL? It does resolve and thus let you locate a resource, no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I know, URL is only used for web resources.
@rflechtner I'm merging the PR since I will build on top of this to tackle https://github.com/KILTprotocol/ticket/issues/2093. |
* feat: static copy of DID json-ld contexts * test: update tests * chore: use shorter ipfs URL
fixes https://github.com/KILTprotocol/ticket/issues/1999
Adds static copies of json-ld context files relevant to KILT DID documents to the sdk, as well as the respective URI to DID documents exported to the conformant json-ld representation.
How to test:
n/a
Checklist: