-
Notifications
You must be signed in to change notification settings - Fork 77
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
subscriptions: add container hash to notification event #2192
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2192 +/- ##
==========================================
+ Coverage 83.72% 83.77% +0.04%
==========================================
Files 296 297 +1
Lines 28319 28347 +28
==========================================
+ Hits 23711 23747 +36
+ Misses 3259 3242 -17
- Partials 1349 1358 +9
Continue to review full report at Codecov.
|
// appExecResultAux is an auxiliary struct for JSON marshalling. | ||
type notificationEventAux struct { |
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.
Comment is not updated.
pkg/rpc/response/events.go
Outdated
// NotaryRequestEvent represents P2PNotaryRequest event either added or removed | ||
// from notary payload pool. |
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.
Comment should also be moved.
External users make use of it. Close #2190.
It is used for subscriptions only, so move it to the subscriptions pkg.
1d134e9
to
5c97e0d
Compare
External users make use of it. Close #2190.