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

add definition for publicKeyHex based on RFC4648 #40

Merged
merged 3 commits into from
Jun 18, 2020

Conversation

kdenhartog
Copy link
Contributor

Signed-off-by: Kyle Den Hartog kyle.denhartog@mattr.global

Signed-off-by: Kyle Den Hartog <kyle.denhartog@mattr.global>
@kdenhartog kdenhartog requested review from msporny and OR13 June 16, 2020 17:06
@OR13 OR13 self-requested a review June 16, 2020 17:11
@OR13
Copy link
Collaborator

OR13 commented Jun 16, 2020

We need this to be defined in a JSON-LD Context... since v3-unstable is currently blocked, how can we add things like this... @msporny ?

Copy link
Collaborator

@OR13 OR13 left a comment

Choose a reason for hiding this comment

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

This property MUST be defined in security-vocab JSON-LD as well.... the question is which context.

@msporny
Copy link
Collaborator

msporny commented Jun 16, 2020

since v3-unstable is currently blocked, how can we add things like this... @msporny ?

Why is v3-unstable blocked?

@kdenhartog
Copy link
Contributor Author

Yup, that's the part I wasn't sure about if it should be added to v2 or v3. Will add to this PR before merging.

Copy link
Collaborator

@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.

Minor nits, otherwise LGTM.

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
@OR13
Copy link
Collaborator

OR13 commented Jun 16, 2020

since v3-unstable is currently blocked, how can we add things like this... @msporny ?

Why is v3-unstable blocked?

#35

I suppose we can define the file again in this PR.

@kdenhartog
Copy link
Contributor Author

kdenhartog commented Jun 16, 2020

since v3-unstable is currently blocked, how can we add things like this... @msporny ?

Why is v3-unstable blocked?

#35

I suppose we can define the file again in this PR.

Yeah I can do it that way and then we'll just need to resolve the merge conflicts on #35 after. At least then we get v3 started as well.

Added this in the latest commit @OR13

Signed-off-by: Kyle Den Hartog <kyle.denhartog@mattr.global>
@OR13 OR13 merged commit 8c306ef into w3c-ccg:master Jun 18, 2020
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.

4 participants