-
Notifications
You must be signed in to change notification settings - Fork 2
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
Transfer proof #121
Transfer proof #121
Conversation
Here's the POC for handling challenges https://github.com/nevermined-io/data-transfer-poc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@@ -16,16 +16,19 @@ jobs: | |||
- uses: actions/checkout@v2 | |||
- uses: actions/setup-node@v1 | |||
with: | |||
node-version: '10.x' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about changing the node version here. Previous experiences bumping to node >=12.x created some issues.
Is node >=14.x a requirement?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, looks like snarkjs and circom depend on it...
Description
Implement the condition and agreement for access with transfer proof.
Currently the condition is pretty generic, so just access to some data perhaps should be connected to DID hash or something?
Is this PR related with an open issue?
Types of changes
Checklist:
Funny gif