-
Notifications
You must be signed in to change notification settings - Fork 54
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.
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
Fix/did web create key #430
Conversation
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.
thank you!
Codecov Report
@@ Coverage Diff @@
## main #430 +/- ##
=======================================
Coverage 56.95% 56.95%
=======================================
Files 67 67
Lines 7334 7334
=======================================
Hits 4177 4177
Misses 2419 2419
Partials 738 738
|
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.
Thanks for fixing! Would you mind adding a test that failed before these changes were made?
Yup I could do that. |
Added tests, let me know if you'd like me to modify it or add to it in any way to enhance overall testing. This is what it looked like failing on main. |
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.
Amazing thanks!
74d7d03
to
b80a228
Compare
Rebased main and pushed. |
* Did Web Validation * lint and test fix * content type const --------- Co-authored-by: Gabe <[email protected]>
* Fixed resolution problems. * Add canonical ID when applying patch changes to a did doc * ptr
* Added docs and fixed required fields for resolution * Nicer lines
* update vcs * fix lint * update go version * pr comments * pr comments
sorry @LiranCohen can you rebase once more and I'll merge? |
merging in #430 |
Was fiddling with did:web and noticed that the owner key ID and Controller were mismatched within the CreateDoc function.
Searched around and saw that it was also mismatched on the did:peer resolve method.