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

Intro editorial review #111

Merged
merged 2 commits into from
Nov 27, 2019
Merged

Intro editorial review #111

merged 2 commits into from
Nov 27, 2019

Conversation

grantnoble
Copy link

@grantnoble grantnoble commented Nov 18, 2019

Some wording changes, some punctuation, spacing, and line length updates.


Preview | Diff

Some wording changes, some punctuation, and line length updates.
Copy link
Contributor

@talltree talltree left a comment

Choose a reason for hiding this comment

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

Great job, Grant. Thanks much for doing this.

index.html Outdated
designed to enable the controller of a <a>DID</a> to prove control over
it and to be implemented independently of any centralized registry, identity
provider, or certificate authority. <a>DIDs</a> are URLs that relate a
<a>DID subject</a> allowing trustable interactions with that subject by way of a
Copy link
Member

Choose a reason for hiding this comment

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

It now doesn't say what the DID subject is related to. Maybe

Suggested change
<a>DID subject</a> allowing trustable interactions with that subject by way of a
<a>DID subject</a> to a DID document, allowing trustable interactions with that subject.

Copy link
Author

Choose a reason for hiding this comment

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

Updated. Thanks!

<a>DIDs</a>. This creates an interoperability bridge between the worlds of
centralized, federated, and <a>decentralized identifiers</a>.
</p>
<p class="note">
Copy link
Member

Choose a reason for hiding this comment

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

👍

@rhiaro rhiaro added the editorial Editors should update the spec then close label Nov 18, 2019
Copy link
Member

@msporny msporny left a comment

Choose a reason for hiding this comment

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

Editorial, multiple positive reviews, no objections, merging.

@msporny msporny merged commit f2c6951 into w3c:master Nov 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial Editors should update the spec then close
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants