ceramicnetwork / js-did

A simple interface to interact with DIDs that conform to the DID-provider interface.
Other
96 stars 28 forks source link

add tezos support #127

Closed Jackey351 closed 1 year ago

Jackey351 commented 2 years ago

Add Tezos support

Description

0xqibing commented 1 year ago

Hi, @zachferland I sent you some figures in Discord, including signature request and console results. You can also make a test using Template Wallet and run the demo. Note that we only support tz1 account for now (ed25519), because it is enough for web users. Template uses tz1 as the default account, and it is the best Chrome wallet now (I cannot find a second usable version). It is easy to add support for tz2 and tz3 when we can test it.

zachferland commented 1 year ago

Thanks for the the changes, helper methods look good now and docs and everything else does as well. Think its good with just tz1 if that is mostly typically used. I might spend some time to try it out, but should be good to merge/release beginning of next week.

0xqibing commented 1 year ago

Hi, @zachferland. For the Template wallet, it did not recognize the '\n', so the first look of SIWE message is not that formatted. I think it depends on the wallet, maybe another works.

zachferland commented 1 year ago

Yeah partly up to wallets and standards in ecosystem on how to display messages like this, think it makes sense to keep this way (with new line breaks), other ecosystem wallets support it

Merging, and will release a first version

Thanks again for adding tezos support! 🚀🎉

zachferland commented 1 year ago

Actually, a few build issues, will see if i can fix quickly first

zachferland commented 1 year ago

Just lock file, going to merge and update lock file on main