Modify sample to use did:dht instead of did:ion #22
Conversation
kirahsapong
requested review from
michaelneale,
mistermoe and
phoebe-lew
as code owners
| @@ -39,11 +39,11 @@ export const config: Config = { | |||
| // a new one will be generated every time the process starts. | |||
| if (!config.did) { | |||
| console.log('Creating an ephemeral DID.....') | |||
| const DidIon = await DidIonMethod.create({ | |||
| const DidDht = await DidDhtMethod.create({ publish: true, | |||
There was a problem hiding this comment.
it will publish by default, so no need for publish: true, unless you want to make it explicit
There was a problem hiding this comment.
getting the same error mentioned here without passing publish: true - currently on 0.2.4 (same as tbdex) sounds like that is removed in future versions?
There was a problem hiding this comment.
confirmed! 0.2.4 defaults to false, later versions default to true. will need to update tbdex before we can update here!
| @@ -5,7 +5,7 @@ import fs from 'node:fs' | |||
|
|
|||
| import 'dotenv/config' | |||
|
|
|||
| import { DidIonMethod, PortableDid } from '@web5/dids' | |||
| import { PortableDid, DidDhtMethod } from '@web5/dids' | |||
There was a problem hiding this comment.
This will change soon, but good stuff for now 👍
There was a problem hiding this comment.
noice! yee once tbdex updates will update here tew
|
nice - could the example scripts also be converted to use it? should they? ie does https://github.com/TBD54566975/tbdex-pfi-exemplar/blob/main/src/example/utils.ts - automatically use diddht? |
@michaelneale done! |
closes #6