-
Notifications
You must be signed in to change notification settings - Fork 34
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
Content Event Online #9930 #9941
base: master
Are you sure you want to change the base?
Conversation
final ImmutableMap.Builder<String, String> builder = ImmutableMap.<String, String>builder() | ||
.put( "id", content.getId().toString() ) | ||
.put( "path", content.getPath().toString() ) | ||
.put( "repo", ContextAccessor.current().getRepositoryId().toString() ); |
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.
don't use context fore repoId
.put( "repo", ContextAccessor.current().getRepositoryId().toString() ); | ||
|
||
final ContentPublishInfo publishInfo = content.getPublishInfo(); | ||
if ( publishInfo != null ) |
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.
this is just for example. Not needed in final version
Codecov ReportBase: 84.04% // Head: 84.02% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #9941 +/- ##
============================================
- Coverage 84.04% 84.02% -0.03%
+ Complexity 19272 19261 -11
============================================
Files 2583 2583
Lines 67729 67715 -14
Branches 5449 5441 -8
============================================
- Hits 56926 56897 -29
- Misses 8090 8115 +25
+ Partials 2713 2703 -10
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
No description provided.