-
Notifications
You must be signed in to change notification settings - Fork 10
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
Updates re controller property #116
Merged
Merged
Changes from 1 commit
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
a9f97a5
revised language describing the controller property
jandrieu eadc13c
removed extra spaces
jandrieu 44a9c9d
incomplete edits. need to base on new intro opening
jandrieu 76ce24f
revised language describing the controller property
jandrieu dcfc7ac
removed extra spaces
jandrieu e92d99e
minor merge edits
jandrieu 7f54c71
clarified controller property language and discussed subject/controll…
jandrieu 909f01d
Apply suggestions from code review
jandrieu 1f32576
Update index.html
jandrieu 7b8b533
Update index.html
jandrieu b085d30
Update index.html
jandrieu c02cf58
Update index.html
jandrieu 229d7a1
Update index.html
jandrieu 79227a6
Update index.html
jandrieu 8db5053
Update index.html
jandrieu bd6dfd1
Update index.html
jandrieu f8ec6f2
Update index.html
jandrieu 48fa8d4
Update index.html
jandrieu fb4aa43
Update index.html
jandrieu 3e40b39
Update index.html
jandrieu 9c1e39b
merge from upstream
jandrieu 2d6248f
Added definitions for canonical URL and base identifier.
jandrieu 303da6b
Update index.html
jandrieu c4338da
Update index.html
jandrieu 124697a
Update index.html
jandrieu 982605e
Update index.html
jandrieu 515631c
Update index.html
jandrieu ec440ee
Update index.html
jandrieu 17f99f2
Update index.html
jandrieu 13696a2
Update index.html
jandrieu 535f07e
Apply suggestions from code review
jandrieu 7d450be
Update index.html
jandrieu 98f1a59
Update index.html
jandrieu 3e69af1
Update index.html
jandrieu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -928,11 +928,11 @@ <h3>Controllers</h3> | |
<a data-cite="INFRA#string">strings</a>, | ||
each of which conforms to the rules in the [[[URL]]]. | ||
|
||
Each entry in the 'controller property' MUST identify | ||
at least one entity capable of updating the canonical version | ||
of the [=controller document=] which contains the property, such | ||
that subsequent requests for this controller document, from its | ||
canonical location, contain the latest version. | ||
Each entry in the `controller` property MUST identify | ||
an entity capable of updating the canonical version | ||
of the [=controller document=]. | ||
This means that subsequent requests for this controller document, | ||
through its canonical location, would always contain the latest version. | ||
jandrieu marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
If the `controller` property is not present, then control of the document | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. -1 for the reasons stated here: https://github.com/w3c/controller-document/pull/116/files#r1839014775 |
||
is determined entirely by its storage location. | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
-1 for the reasons stated here: https://github.com/w3c/controller-document/pull/116/files#r1839014775