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.

Sign up for GitHub

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

Jump to bottom

Replace usage of did:ion in tests with another DID method #218

Closed
mistermoe opened this issue Feb 5, 2024 · 1 comment · Fixed by #248 or #262
Closed

Replace usage of did:ion in tests with another DID method #218

mistermoe opened this issue Feb 5, 2024 · 1 comment · Fixed by #248 or #262
Assignees
@jiyoonie9

Comments

@mistermoe
Copy link
Member

because we're deprecating use of ION
@mistermoe mistermoe added the bug Something isn't working label Feb 5, 2024
@mistermoe mistermoe removed the bug Something isn't working label Feb 5, 2024
@mistermoe mistermoe mentioned this issue Feb 5, 2024
Closed
grahnj added a commit to grahnj/web5-kt that referenced this issue Feb 9, 2024
@grahnj
Removed DidIon references from non DidIon tests and replaced with DidDht
28816a4 
* Fixes issue TBD54566975#218
* Added DidDht integration test (left remaining DidIon test for removal when DidIon is fully removed)
* Migrated InvalidStatusException to did exceptions package
* Replaced DidIon.create in tests with DidDht.create
* Replaced DidIon.load in tests with DidDht.load
@grahnj grahnj mentioned this issue Feb 9, 2024
Closed
1 task
@jiyoonie9 jiyoonie9 linked a pull request Mar 11, 2024 that will close this issue
Removing did-common-java dependency #248
Merged
4 tasks
@jiyoonie9
Copy link
Contributor

This work was done in all packages except credential package. Will work on defaulting to did:dht in credential package tests in the PR #262

@jiyoonie9 jiyoonie9 self-assigned this Mar 11, 2024
@jiyoonie9 jiyoonie9 linked a pull request Mar 12, 2024 that will close this issue
Implementing BearerDid / PortableDid, exposing our own JWS, JWT, JWK types #262
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jiyoonie9 jiyoonie9

Labels
None yet
Projects
SDK Development
Status: Done
Milestone
No milestone
4 participants
@andresuribe87 @mistermoe @nitro-neal @jiyoonie9