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

Support Full DID Resolution: options, full result, etc. #331

Open
decentralgabe opened this issue Mar 28, 2023 · 1 comment
Open

Support Full DID Resolution: options, full result, etc. #331

decentralgabe opened this issue Mar 28, 2023 · 1 comment
Assignees
@andresuribe87
Labels
dids

Comments

@decentralgabe
Copy link
Member 

          Whenever I see `any`, it's typically a symptom of an API that hasn't been fleshed out yet. In this case, it's unclear the shape of what `ResolutionOption` is acceptable by the functions that use it.

Instead, I would define an interface. Mind creating an issue to address?

Separately, This doesn't seem to be used anywhere.

Originally posted by @andresuribe87 in #329 (comment)
@decentralgabe decentralgabe self-assigned this Mar 28, 2023
@decentralgabe decentralgabe mentioned this issue Mar 28, 2023
Merged
@decentralgabe decentralgabe changed the title Utilize DID Resolution Options Support Full DID Resolution: options, full result, etc. May 1, 2023
@decentralgabe decentralgabe mentioned this issue May 1, 2023
Merged
@decentralgabe
Copy link
Member Author

As per https://github.com/TBD54566975/ssi-sdk/pull/363/files#r1181715361

need to support resolution results for all DID resolvers

@decentralgabe decentralgabe mentioned this issue May 24, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andresuribe87 andresuribe87

Labels
dids
Projects
No open projects
SSI
Status: 📋 Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@andresuribe87 @decentralgabe