search

w3c / did-core

W3C Decentralized Identifier Specification v1.0
https://www.w3.org/TR/did-core/
Other
405 stars 94 forks source link

Fix editorial issues in Core Properties section. #630

Closed msporny closed 3 years ago

msporny commented 3 years ago

Preview | Diff

msporny commented 3 years ago

@iherman wrote:

However. The change also removed the note whereby in the DID model, the order of the items in an ordered set are irrelevant. I just checked and the note was the only place in the document where this fact was mentioned. This note should be reinstated in the text.

The statement was moved to a separate section in the specification in PR #631 -- see https://github.com/w3c/did-core/pull/631/files#diff-0eb547304658805aad788d320f10bf1f292797b5e6d745a3bf617584da017051R1276

PS: I'm really glad that reviewers are paying attention to and are catching these potential oversights. :)

iherman commented 3 years ago

The statement was moved to a separate section in the specification in PR #631 -- see https://github.com/w3c/did-core/pull/631/files#diff-0eb547304658805aad788d320f10bf1f292797b5e6d745a3bf617584da017051R1276

PS: I'm really glad that reviewers are paying attention to and are catching these potential oversights. :)

Sigh... Such tricks make our life difficult 😄

I am fine merging this PR and #631 together... 😉

msporny commented 3 years ago

@iherman wrote:

Such tricks make our life difficult smile

Just making sure everyone is paying attention... :P

msporny commented 3 years ago

Editorial, multiple reviews, changes requested and made, no objections, merging.