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

FIP: Soulbound claims #49

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

FIP: Soulbound claims #49

wants to merge 6 commits into from

Conversation

molekilla
Copy link

No description provided.

@molekilla molekilla requested a review from nugaon June 28, 2022 17:12
Copy link
Collaborator

@nugaon nugaon left a comment

Choose a reason for hiding this comment

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

For me it seems more like a bounty description than a FIP.
The document misses many parts that an Improvement Proposal should hold IMO, such as interface descriptions (for JS libraries and smart contracts), elaborating the required business logic, provide an exact solution for verification proofs, explaining how to use ZKP here...

reading this document I have more questions than answers that should be the other way around.


# Summary

Soulbound tokens for identity claims allows for a better user and developer experience by using existing Web3 technologies based on NFT token standards. In this FIP we described our findings and recommendations for FDP identity claims.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Soulbound tokens are not NFTs, maybe non-transferable NFTs. one can get the wrong assumption about SBTs based on this IMO.

Copy link
Author

Choose a reason for hiding this comment

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

the sample smart contract is a NFT without transfer solidity code. but terminology wise is not the same

- As an issuer, I need to revoke claims
- As a holder, I need to read my issued claims
- As a holder, I need to share my issued claims with a verifier
- As a verifier, I need to ask a holder for verification
Copy link
Collaborator

Choose a reason for hiding this comment

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

this process should support automatic steps without async, shouldn't it?

Copy link
Author

Choose a reason for hiding this comment

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

Automatic steps you refer to delegated authority (EOA or smart contract address
) like did-ethr? I didn't described this here because it might be a separate topic


## Milestone 1 - Issuer

A Dapp with issuer capabilities can grant and revoke SBT to holders. The claims that the issuers creates are stored in Swarm, with the hash in the onchain SBT token.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
A Dapp with issuer capabilities can grant and revoke SBT to holders. The claims that the issuers creates are stored in Swarm, with the hash in the onchain SBT token.
A Dapp with issuer capabilities can grant and revoke SBT to holders. The claims that the issuers create are stored on Swarm, with the hash of the on-chain SBT token.

The implementation must be able to:

- Have SBT issuer smart contracts
- JavaScripts / Typescript library
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would link here the JS library for FDP contracts for reference: https://github.com/fairDataSociety/fdp-contracts/tree/master/js-library

Copy link
Author

Choose a reason for hiding this comment

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

Regarding more details, you are very right, that's the reason I wanted to get your feedback. Indeed, it might be technical bounty, but I need to speced it out

@molekilla molekilla requested a review from nugaon July 8, 2022 22:37
@crtahlin crtahlin added the documentation Improvements or additions to documentation label Aug 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants