Skip to content

@Context #557

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

Closed
nadalin opened this issue Apr 18, 2019 · 5 comments
Closed

@Context #557

nadalin opened this issue Apr 18, 2019 · 5 comments
Labels
pending close Close if no objection within 7 days
Milestone

Comments

@nadalin
Copy link

nadalin commented Apr 18, 2019

"The value of the @context property MUST be an ordered set where the first item is a URI with the value https://www.w3.org/2018/credentials/v1."

I don't believe that this has to be a ordered set, the specification should allowed this to be treated as just a string if this value has to be present, which I still have concerns about

@David-Chadwick
Copy link
Contributor

This issue has already been raised by me in PR #548. So I suggest that when it is answered there, this issue can be closed.

@nadalin
Copy link
Author

nadalin commented Apr 18, 2019

@David-Chadwick Agree

@burnburn burnburn added this to the CR-Exit milestone Apr 25, 2019
@msporny
Copy link
Member

msporny commented Apr 26, 2019

I don't believe that this has to be a ordered set, the specification should allowed this to be treated as just a string if this value has to be present, which I still have concerns about

It has to be an ordered set. If it is not, JSON implementations would be forced to use a JSON-LD processor, which is not what you want.

The reason is elaborated upon in the last paragraph of this section (which was added based on changes that you requested): https://w3c.github.io/vc-data-model/#semantic-interoperability

@msporny
Copy link
Member

msporny commented Apr 26, 2019

This issue has already been raised by me in PR #548. So I suggest that when it is answered there, this issue can be closed.

PR #548 was merged with PR #546, which was approved by the WG and merged into the specification, thus addressing this issue.

@stonematt
Copy link
Contributor

From VCWG call on April 30, 2019:
RESOLUTION: Issue #557 is addressed by PR #546 which made non-substantive changes to explain why @context is an ordered set. PR #546 has been merged and issue #557 should be closed.

@stonematt stonematt added the pending close Close if no objection within 7 days label Apr 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending close Close if no objection within 7 days
Projects
None yet
Development

No branches or pull requests

5 participants