Skip to content
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

Update JSON Processing section to Credential-Type-Specific Processing. #1351

Merged
merged 4 commits into from
Nov 25, 2023

Conversation

msporny
Copy link
Member

@msporny msporny commented Nov 14, 2023

This PR attempts to address issue #1290 by applying the result of the VCWG poll to rename the section to "Credential-type-specific Processing". Here are the results of the poll:

image

It also updates the language in an attempt to eliminate the use of "JSON Processing" and replace it with something that the VCWG, and @awoie -- specifically, will hopefully find more palatable.


Preview | Diff

index.html Outdated Show resolved Hide resolved
Co-authored-by: Dave Longley <dlongley@digitalbazaar.com>
index.html Outdated Show resolved Hide resolved
@iherman
Copy link
Member

iherman commented Nov 15, 2023

The issue was discussed in a meeting on 2023-11-15

  • no resolutions were taken
View the transcript

1. poll results.

Manu Sporny: We should review the results from the poll.
… Perhaps people could emote here to add late votes?

Joe Andrieu: what's the URL for the poll?

Dave Longley: https://www.opavote.com/en/vote/5254957337935872.

Manu Sporny: The context is that we decided a few weeks ago to run a poll. We wanted to change the name of a certain aspect. It looks like we'll choose to 'Credential Type-specific' processing.

Orie Steele: Of these options, "Credential Type" is probably the best, but it omits the fact that the context can change information regardless of credential type.

Manu Sporny: I have already created a PR. I suggest that we close the poll and I can modify the PR to show the chosen result.

Dave Longley: Orie: we should mention immutable or "semantically immutable" contexts better in the section.

See github pull request vc-data-model#1351.

See github issue vc-data-model#1290.

Orie Steele: There was also a thread in the W3C CCG on this topic recently... https://lists.w3.org/Archives/Public/public-credentials/2023Nov/0030.html.

Joe Andrieu: I appreciate the term 'limited' but I don't like the term 'unlimited'. I think that could be a problem.

Joe Andrieu: +1 to get us unstuck.

Manu Sporny: The poll is not a binding vote, so we can still take into account other views.

Joe Andrieu: I'll add here (so the meeting can move on) that I also don't think the distinction is valid. It isn't a choice between applications that can work with any credential or those with specific credentials. =(.

Brent Zundel: It is OK to have a compromise, but I would like to avoid spending further time on the topic during this meeting.

Co-authored-by: Dave Longley <dlongley@digitalbazaar.com>
Copy link
Member

@TallTed TallTed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully improving clarity...

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
@msporny
Copy link
Member Author

msporny commented Nov 25, 2023

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

@msporny msporny merged commit 352bb29 into main Nov 25, 2023
1 check passed
@msporny msporny deleted the msporny-processing-retitle branch November 25, 2023 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants