-
Notifications
You must be signed in to change notification settings - Fork 204
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
support newer did-communication service type #233
support newer did-communication service type #233
Conversation
Signed-off-by: Timo Glastra <[email protected]>
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.
Happy to see we will be up to date 👍 We should just align naming conventions.
Signed-off-by: Timo Glastra <[email protected]>
Signed-off-by: Timo Glastra <[email protected]>
@jakubkoci I updated the PR according to your feedback |
Git ignores casing by default. Signed-off-by: Timo Glastra <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #233 +/- ##
==========================================
- Coverage 89.06% 89.06% -0.01%
==========================================
Files 174 178 +4
Lines 3302 3383 +81
Branches 364 389 +25
==========================================
+ Hits 2941 3013 +72
- Misses 358 367 +9
Partials 3 3
Continue to review full report at Codecov.
|
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.
👍
…tion#233) Signed-off-by: Timo Glastra <[email protected]>
This adds support for the newer
did-communication
service type in did docs according to RFC 0076IndyAgent services are still supported and we now send out both. This means it is backwards compatible in both ways (receiving and sending diddocs)