Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

feat: out of band messaging #1436

Merged
merged 1 commit into from
Mar 9, 2020

Conversation

sudeshrshetty
Copy link
Contributor

@sudeshrshetty sudeshrshetty commented Mar 6, 2020

Signed-off-by: sudesh.shetty sudesh.shetty@securekey.com

@sudeshrshetty sudeshrshetty changed the title WIP out of band messaging feat: out of band messaging Mar 9, 2020
@codecov
Copy link

codecov bot commented Mar 9, 2020

Codecov Report

Merging #1436 into master will decrease coverage by <.01%.
The diff coverage is 97.61%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1436      +/-   ##
==========================================
- Coverage   91.28%   91.28%   -0.01%     
==========================================
  Files         139      139              
  Lines        9228     9257      +29     
==========================================
+ Hits         8424     8450      +26     
- Misses        455      457       +2     
- Partials      349      350       +1
Impacted Files Coverage Δ
pkg/controller/rest/messaging/operation.go 100% <ø> (ø) ⬆️
pkg/didcomm/messenger/messenger.go 90.9% <100%> (+0.66%) ⬆️
pkg/controller/command/messaging/command.go 97.31% <97.22%> (-1.47%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 70fe0cc...8292347. Read the comment docs.

.golangci.yml Outdated
@@ -71,7 +71,7 @@ linters-settings:
- unnamedResult
- whyNoLint # TODO enable.
funlen:
lines: 60
lines: 100
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
lines: 100
lines: 60

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed it

Copy link
Contributor

@llorllale llorllale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

- message service command controller to use messanger api
- send message to support out of band messages
- Part of hyperledger-archives#1058

Signed-off-by: sudesh.shetty <sudesh.shetty@securekey.com>
@fqutishat fqutishat merged commit 761b437 into hyperledger-archives:master Mar 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants