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

The WG may produce registries #98

Merged
merged 8 commits into from
Mar 9, 2022
Merged

The WG may produce registries #98

merged 8 commits into from
Mar 9, 2022

Conversation

brentzundel
Copy link
Member

@brentzundel brentzundel commented Mar 8, 2022

This PR takes one part of PR #85 which I hope will achieve consensus.


Preview | Diff

Copy link
Member

@msporny msporny left a comment

Choose a reason for hiding this comment

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

I would personally like to be more specific, but would be fine w/ this text as well.

@brentzundel

This comment was marked as resolved.

@brentzundel brentzundel requested a review from msporny March 8, 2022 18:19
index.html Outdated
Comment on lines 313 to 315
to support extension points in the above normative deliverables. Registries for
extension points that are required by any of the above normative deliverables
must have at least one standardized entry.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
to support extension points in the above normative deliverables. Registries for
extension points that are required by any of the above normative deliverables
must have at least one standardized entry.
to support extension points in the above normative deliverables. Registries for
extension points that are required by any of the above normative deliverables
must have at least one standardized entry.

-1, I expect that we'd have to remove features in the VC spec and that would be controversial.

Copy link
Member

@msporny msporny Mar 8, 2022

Choose a reason for hiding this comment

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

Hrm, on second read, I'm confused. So, existing extension points that don't have normative registry entries would still be allowed?

Copy link
Member Author

Choose a reason for hiding this comment

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

I tried to add the language that seemed to have the most consensus during the WG call on March 2nd.
Currently, the only normatively required extension point in the VC data model is 'proof', which we will standardize in the data integrity work. I don't see any features that we would need to remove. What am I missing?

Copy link
Member

Choose a reason for hiding this comment

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

I don't see any features that we would need to remove. What am I missing?

We currently don't have normative deliverables in the current charter defined for any of the following extension points:

https://www.w3.org/TR/vc-data-model/#status
https://www.w3.org/TR/vc-data-model/#evidence
https://www.w3.org/TR/vc-data-model/#terms-of-use
https://www.w3.org/TR/vc-data-model/#refreshing
https://www.w3.org/TR/vc-data-model/#data-schemas

One reading of the language wrt. "normative registry entries" is that every single one of those features should be removed in the VC 2.0 work, which I expect would lead to objections from organizations relying on those extension points out in the field.

Copy link
Member Author

Choose a reason for hiding this comment

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

none of those are required by the VC data model, therefore none of those would need a standardized entry in a registry.
But also, the language has been removed form this PR and moved to #101

Copy link
Member

Choose a reason for hiding this comment

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

none of those are required by the VC data model, therefore none of those would need a standardized entry in a registry.

Ah, I see, and am now slowly remembering the conversation... I'm not opposed IF everyone reads the language in the same way. I'll try providing some changes in PR #101 to try to make it clear that it only applies to normative properties... though, I expect, this wouldn't address @jyasskin's concern.

Copy link
Member

@msporny msporny left a comment

Choose a reason for hiding this comment

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

I was supportive of this PR, but now am not given the recent change in 6753b9e and confusion in #98 (comment).

@brentzundel
Copy link
Member Author

@msporny since the original intent of this PR was to try and find consensus on the WG having any registries, I have removed the controversial change and will reintroduce it in another PR

Copy link

@selfissued selfissued left a comment

Choose a reason for hiding this comment

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

This is great! This is all we need to say about registries in the charter.

@brentzundel brentzundel merged commit 8814075 into w3c:main Mar 9, 2022
@iherman
Copy link
Member

iherman commented Mar 9, 2022

The issue was discussed in a meeting on 2022-03-09

  • no resolutions were taken
View the transcript

1.2. The WG may produce registries (pr vc-wg-charter#98)

See github pull request vc-wg-charter#98.

Brent Zundel: looks like we have rough agreement, we want registries in the charter for the most part.
… can you throw your approval instead of the stale CR, any opposition to merging this PR.

Joe Andrieu: I oppose, for reasons stated.

Brent Zundel: Joe your opposition is noted, and will go into the comments on the PR.

Oliver Terbu: does registries include data integrity proof registry, my concern is it should be permissive enough, and if we can get that right then I'm on board.

Manu Sporny: +1 to registry not being restrictive... it needs to be permissive..

Brent Zundel: the charter doesn't restrict as of yet but the vcwg may restrict at a later date.
… over the noted objection from joe will merge the PR.
… two other PRs that build towards what 85 was aiming for, looking at 99 next..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants