-
Notifications
You must be signed in to change notification settings - Fork 11
Add example that does not contain @context.
#124
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
Conversation
1285255 to
8058ef8
Compare
dlongley
left a comment
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.
See suggestion to make the example valid.
Co-authored-by: Dave Longley <dlongley@digitalbazaar.com>
|
It would be helpful to the reader to provide a description of the controller document underneath it such as: |
|
@David-Chadwick wrote:
I agree, and we do this in multiple sections throughout the specification... when we introduce a new property, we provide an example and explain what the property does. If we added that text here, we'd re-state things stated earlier in the specification. I do also admit that we need to make another editorial pass on the examples to make sure they're properly explained, as @David-Chadwick notes. I've raised an issue to track that concern here: #125 |
|
The issue was discussed in a meeting on 2024-11-20
View the transcript1.6. Add example that does not contain
|
| (`https://www.w3.org/ns/did/v1`). | ||
| </p> | ||
|
|
||
| <pre class="example nohighlight" title="A controller document without an @context property"> |
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.
| <pre class="example nohighlight" title="A controller document without an @context property"> | |
| <pre class="example nohighlight" title="A controller document without a @context property"> |
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.
I generally read @context as "at-context", not as "context", partly because it is distinct from context. That reading makes an @context ("an at-context") appropriate, not a @context ("a at-context"). Perhaps others should weigh in, so we can decide which indefinite article to use throughout.
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.
I tend toward TallTed's perspective. I have a similar inner voice that pronounces the @: "at-context".
But I think it's more important to figure out one way and be consistent.
A bit of sleuthing shows @selfissued's take shows up once in the JSON-LD spec, but the spec also uses "an @context" six times.
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.
@jandrieu — Thanks for that sleuthing. w3c/json-ld-syntax#448 created to make that errant a match the other six an.
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.
I also read it as "an at-context"
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.
Yes, the Editors have attempted to consistently read @context as at context and therefore try to keep a consistent rule of us an vs. a.
|
Editorial, multiple reviews, changes requested and made, no objections, merging. |
This PR is an attempt to address issue #54 by adding an example that does not contain an
@contextproperty.Preview | Diff