-
Notifications
You must be signed in to change notification settings - Fork 55
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
Updated section "Token mappings" #1075
Conversation
Updated normative text on how to use the `Link` header Removed Note about "Alternate ways of conveying URI / token mappings" as per #1064
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.
Reviewed and confirmed technically accurate.
As an editorial note, I wonder whether we need the concept of a "token / URI mapping". We might be able to write down the same thing by just referring to tokens. Will add optional suggestion as comment.
need to provide a <a>token</a> / URI mapping via this <strong>list profiles</strong> function. In this | ||
HTTP functional profile of this specification, this may be implemented in a number of ways. The preferred | ||
way is to add "token" parameters to the HTTP Content-Profile and Link HTTP headers. | ||
need to provide a <a>token</a> / URI mapping via this <strong>list profiles</strong> function. |
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.
We could write this instead as "needs to indicate for every supported token to which profile URI it corresponds". This eliminates the concept of a "token / URI mapping" (and associated questions such as whether it needs to be complete, injective, etc.).
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.
fixed a couple of small HTML things, now all approved
Updated normative text on how to use the
Link
headerRemoved Note about "Alternate ways of conveying URI / token mappings" as per #1064
Preview at https://raw.githack.com/w3c/dxwg/larsgsvensson-update-token-mappings/conneg-by-ap/index.html#listprofiles-tokens
Since this introduces normative text, please review carefully and also please don't merge until we've had @RubenVerborgh's review