-
Notifications
You must be signed in to change notification settings - Fork 158
refactor: VDR support for deactivate/update #2473
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2473 +/- ##
==========================================
+ Coverage 89.22% 89.24% +0.01%
==========================================
Files 253 253
Lines 18495 18517 +22
==========================================
+ Hits 16503 16525 +22
Misses 1173 1173
Partials 819 819
Continue to review full report at Codecov.
|
a1bb153
to
9c99646
Compare
func (v *VDR) Deactivate(did string, opts ...vdrapi.DIDMethodOption) error { | ||
return fmt.Errorf("not supported") | ||
} | ||
|
||
// Accept did method. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How does VDR indicate that it can handle only resolution vs the full interface?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need to consider having two sets of VDRs loaded into the framework:
- resolvers
- writers / full lifecycle
and make the writer interface a superset of the resolver interface.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
c107224
to
d0b6dc3
Compare
closes hyperledger-archives#2423 Signed-off-by: Firas Qutishat <firas.qutishat@securekey.com>
closes #2423
Signed-off-by: Firas Qutishat firas.qutishat@securekey.com