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

Terms of use #188

Merged
merged 2 commits into from
Jul 24, 2018
Merged

Terms of use #188

merged 2 commits into from
Jul 24, 2018

Conversation

David-Chadwick
Copy link
Contributor

@David-Chadwick David-Chadwick commented Jun 19, 2018

Rebased original PR#170 and addressed additional comments made to it


Preview | Diff

@stonematt
Copy link
Contributor

Fixes #108

@stonematt
Copy link
Contributor

@David-Chadwick both of these examples are prohibitions. Can we include an example that grants usage. I'm thinking this might be a course to solve the time to live issue #164

The issuer could issue this VC for use until dateTime:x at which time the verifier must update from the renewal service?

@David-Chadwick
Copy link
Contributor Author

@stonematt I think you might find that all terms of use are actually prohibitions, as they are there to limit the use of the VC in some respect. So useUntil is a prohibition of the VC being available forever.

@David-Chadwick
Copy link
Contributor Author

@msporny @dlongley . What is holding up your review?

Copy link
Contributor

@dlongley dlongley left a comment

Choose a reason for hiding this comment

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

LGTM other than a typo nitpick.

@dlongley
Copy link
Contributor

What is holding up your review?

Vacation! 😄

@burnburn burnburn merged commit f037b70 into w3c:gh-pages Jul 24, 2018
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