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

Added related resource text and example #177

Merged
merged 4 commits into from
Jul 25, 2023
Merged

Added related resource text and example #177

merged 4 commits into from
Jul 25, 2023

Conversation

andresuribe87
Copy link
Collaborator

@andresuribe87 andresuribe87 commented Jul 17, 2023

Fixes #120


Preview | Diff

index.html Outdated Show resolved Hide resolved
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
Copy link
Collaborator

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.

Copy link
Collaborator Author

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?

Copy link
Member

@TallTed TallTed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removes redundancy

index.html Outdated Show resolved Hide resolved
Co-authored-by: Ted Thibodeau Jr <[email protected]>
@andresuribe87
Copy link
Collaborator Author

@TallTed do you have any additional concerns that I can address in this PR?

Copy link
Member

@TallTed TallTed left a 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

index.html Outdated Show resolved Hide resolved
Co-authored-by: Ted Thibodeau Jr <[email protected]>
@decentralgabe decentralgabe merged commit 2743534 into main Jul 25, 2023
1 check passed
@decentralgabe decentralgabe deleted the issue_120 branch July 25, 2023 15:46
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.

Consider adding language on schema immutability
4 participants