Switch over from local storage implementation to Ceramic for VC storage

passportxyz / passport

Passport allows users to prove their identity through a secure, decentralized UI

Other

955 stars 451 forks source link

Switch over from local storage implementation to Ceramic for VC storage #36

Closed brentmartin closed 2 years ago

brentmartin commented 2 years ago

GIVEN I have verified a stamp in one browser AND I have not completed that stamp in a second browser WHEN I visit my passport page in a new browser THEN I still see my verified stamp

Notes

adding ceramic to CI/CD (can be its own story/card)
Implementing/replacing ceramic as a storage implementation (this card)

Question

what does testing look like?

Story Points 2

brentmartin commented 2 years ago

Notes for PM's:

Look into user flow from Orbis.club
convert this AC into Gherkin

shavinac commented 2 years ago

Dev notes:

Need to remove Vite due to dependency issues with Ceramic - see https://github.com/ChainAgnostic/caip-js/issues/22
Current plan is to refactor app - using NextJS as a base instead

Additional: Reference spike of Passport/Stamp CRUD with Ceramic [using vite + hacky build]

farque65 commented 2 years ago

Dev notes:

Need to remove Vite due to dependency issues with Ceramic - see ESM target is missing ChainAgnostic/caip-js#22

Current plan is to refactor app - using NextJS as a base instead

Addressed by: #45

shavinac commented 2 years ago

For setting up tests

set up a docker-compose script to spin up local node + run integration tests
devs should run integration tests manually and/or run in pre-push hook
later - set up automated integration test in ci/cd?
later / #39 - use remote node instead of local node for testing?