-
Notifications
You must be signed in to change notification settings - Fork 12
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
Refactor "Other Deliverables" section #86
Refactor "Other Deliverables" section #86
Conversation
<li>VC JSON Web Tokens</li> | ||
</ul> | ||
</li> | ||
<li>Extensions for binding multilingual resources for localized user interfaces.</li> |
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 was moved up to line 319.
<li>Test suites for verifiable credential proof types: | ||
<ul> | ||
<li>VC Data Integrity - Ed25519</li> | ||
<li>VC Data Integrity - BBS+</li> | ||
<li>VC JSON Web Tokens</li> |
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.
These were condensed to line 315.
index.html
Outdated
<li>Verifiable Credential Exchange over [GNAP](https://datatracker.ietf.org/doc/html/draft-ietf-gnap-core-protocol)</li> | ||
<li>Verifiable Credential Exchange over <a href="https://datatracker.ietf.org/doc/html/draft-ietf-gnap-core-protocol">GNAP</li> |
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.
Fixing markdown (this isn't a markdown document) -> HTML.
<li>Presentation Request Data Model</li> | ||
<li>Storage and Sharing of Verifiable Credentials</li> | ||
<li>Privacy Guidance for Verifiable Credentials</li> | ||
<li>Extensions for binding multilingual resources for localized user interfaces.</li> | ||
<li>A Developer Guide consisting of one or more notes related to general implementation guidance and best practices for working with VCs, including but not limited to: | ||
<ul> | ||
<li>One or more HTTP protocol definitions for Verifiable Credential Exchange (such as the <a href="https://w3c-ccg.github.io/vc-api/">VC-API</a>)</li> | ||
<li>Guidance on Verifiable Credential Exchange over OpenID Connect</li> |
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.
adding suggestions since there was support to add URLs to OIDC specs when we added GNAP:
<li>Guidance on Verifiable Credential Exchange over OpenID Connect</li> | |
<li>Verifiable Credential Exchange over <a href="https://openid.net/specs/openid-connect-core-1_0.html">OpenID Connect</a>:</li> | |
<ul> | |
<li><a href="https://openid.net/specs/openid-connect-self-issued-v2-1_0.html">Self-Issued OpenID Provider v2</a></li> | |
<li><a href="https://openid.net/specs/openid-connect-4-verifiable-presentations-1_0.html">OpenID Connect for Verifiable Presentations</a></li> | |
<li><a href="https://openid.net/specs/openid-connect-4-verifiable-credential-issuance-1_0.html">OpenID Connect for Verifiable Credential Issuance</a></li> | |
</ul> |
(Happy to do a separate PR, but I think this fits here)
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.
@Sakurann if you are fine raising a separate PR for these changes, we can merge this PR now.
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.
A couple minor tweaks, above
editorial fixes from TallTed Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
@Sakurann please raise a new PR for the changes you recommend in your comment here Merging in accordance with our current working mode. |
Uh oh, the merge was done w/o changing the target branch, so this will never be applied to the main branch. Trying to fix now... |
This PR refactors the "Other Deliverables" section by:
Preview | Diff