-
Notifications
You must be signed in to change notification settings - Fork 8
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
Added related resource text and example #177
Conversation
index.html
Outdated
@@ -441,6 +442,37 @@ <h3>Integrity Validation</h3> | |||
<p class="issue" data-number="143"> | |||
Provide examples for Data Integrity and VC-JWT Credential Schemas | |||
</p> | |||
<p> | |||
The integrity of Credential Schemas of type <a href="#verifiablecredentialschema2023">JsonSchema2023</a> MAY |
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 should be amended to apply to both JsonSchema2023
and VerifiableCredentialSchema2023
, other examples should be updated too.
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.
VerifiableCredentialSchema2023
already includes integrity guarantees being that it's a secured message. For what use cases are you thinking this would be useful?
Co-authored-by: Gabe <[email protected]>
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.
Removes redundancy
Co-authored-by: Ted Thibodeau Jr <[email protected]>
@TallTed do you have any additional concerns that I can address in this PR? |
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.
Just a bit of word order
Co-authored-by: Ted Thibodeau Jr <[email protected]>
Fixes #120
Preview | Diff