-
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
Handling sales with access properly #79
Conversation
So the use case here should be: "As a NFT owner I want to sell them to buyers and give access to DTP encrypted contents" or via 2 DDO services provided by the templates
This needs to be defined by the publisher in the DDO. I think the parameter in the sdk is
Extract an interface for access conditions (i.e
Yep, I think this could be achieved via the 2 services/templates scenario. I think it's cleaner/simpler than 1 template with 4 conditions. |
The reason why the But I think the simplest approach for now is to just use the NFT Sales template instead...
I meant that the node would let the user download assets the same way as their owners, without necessarily processing the |
const [from] = await nevermined.accounts.list() | ||
await plugin.process(params, from, undefined) | ||
return 'success' | ||
} | ||
|
||
@Post('nft-sales-proof') |
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 guess this endpoint will be called by order
method that will be implemented in the sdk-dtp
right?
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 guess at least for now we can use the same end point as normal, basically if there's no access involved it will just be the same thing as normal.
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.
@mrsmkl , I think the best approach would be to use the same endpoint and identify if it's a "regular or DTP" flow. That way we maximize reusing existing code and the endpoints remains the same
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.
@mrsmkl but in case that is a DTP
do we need to implement something in sdk-dtp
right?
Description
Issues that should be thought about
AccessProofCondition
(this will require some work because the args toAccessProofCondition
are just the babyjub keys)Is this PR related with an open issue?
Related to Issue #
Types of changes
Checklist: