web icon indicating copy to clipboard operation
web copied to clipboard

Issue Trust Bonus as a W3C Verifiable Credential to 3Box

Open wyc opened this issue 3 years ago • 8 comments

Description

This change allows users to download their Trust Bonus as a signed W3C Verifiable Credential to their local machine or 3Box profile in addition to minting an NFT. This can then be used to demonstrate their Trust Bonus anywhere supporting the same global data standards. The signing happens with a did-web DID from gitcoin.co, but this component can be further decentralized in the future using ENS + did-eth.

Progress:

  • [x] Implement W3C Verifiable Credential issuance
  • [x] Integrate storage of credentials into 3Box
  • [x] Implement demo application to load credentials from 3Box and verify them
  • [x] Complete tests to hit the endpoint and verify the credential retrieved

A demo deployment with this PR incorporated can be found here: https://gitcoin-demo.spruceid.com/

Tool to retrieve & test the credential: https://demo.spruceid.com/popp/tools/

did-web JSON source: https://gitcoin-demo.spruceid.com/.well-known/did.json

Refers/Fixes

https://github.com/gitcoinco/web/issues/8811

Testing

image

0c157835-8f1f-4c7e-a8f3-abaccf7c4977

wyc avatar Jun 09 '21 18:06 wyc

The issues that were listed above have been resolved in the latest commits. It's ready for another review, especially the doc section, since I wasn't sure about the level of detail to use there.

theosirian avatar Jun 14 '21 15:06 theosirian

Rebased onto current master.

theosirian avatar Jun 14 '21 17:06 theosirian

The code looks alright to me but I'm not very comfortable on adding

  • ts
  • svelte
  • the custom webpack / lint config files

Can this not be done outside this repo and we just end up including the minified file @wyc / @theosirian

cc @octavioamu / @gdixon / @chibie / @zacheryschiller could you guys check this out as well ?


Also from UI I'm not sure this sticks to out styleguide / presskit. @willsputra / @PixelantDesign could you guys check this and drop in feedback ?

thelostone-mc avatar Jun 15 '21 08:06 thelostone-mc

Sure thing, we will move it out of this PR for now. Perhaps a good home for it is https://github.com/gitcoinco/PersonhoodPassport

wyc avatar Jun 15 '21 15:06 wyc

The items mentioned in https://github.com/gitcoinco/web/pull/9032#issuecomment-861277949 have been moved into https://github.com/gitcoinco/PersonhoodPassport/pull/1.

Ready for another review, thanks!

wyc avatar Jun 15 '21 17:06 wyc

Resolved the changes requested by @octavioamu and rebased onto current master.

theosirian avatar Jul 29 '21 19:07 theosirian

Hey @octavioamu - we've since migrated this to Ceramic. We're ready for another review - thanks!

obstropolos avatar Sep 16 '21 04:09 obstropolos

kuvvet itlir

anil2307 avatar Oct 23 '21 09:10 anil2307