(BE) Support VIDEO_60_PERCENT_WATCHED event type #855#423
(BE) Support VIDEO_60_PERCENT_WATCHED event type #855#423mariusmarin-dev wants to merge 5 commits into
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| * @param eventJSON the client supplied event details. | ||
| * @return a {@link SegueErrorResponse} describing the first validation failure, or null if the payload is valid. | ||
| */ | ||
| private SegueErrorResponse validateVideoEngagementEvent(final Map<String, Object> eventJSON) { |
There was a problem hiding this comment.
📝 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> reported by reviewdog 🐶
Abbreviation in name 'eventJSON' must contain no more than '1' consecutive capital letters.
| * @param fieldName the field to read. | ||
| * @return the value as a Double, or null if the field is absent or not a number. | ||
| */ | ||
| private static Double getNumericField(final Map<String, Object> eventJSON, final String fieldName) { |
There was a problem hiding this comment.
📝 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> reported by reviewdog 🐶
Abbreviation in name 'eventJSON' must contain no more than '1' consecutive capital letters.
| for (int i = 1; i < userIds.size(); i++) { | ||
| inParams.append(",?"); | ||
| } | ||
| String inParams = "?" + |
There was a problem hiding this comment.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck> reported by reviewdog 🐶
'+' should be on a new line.
* Refactor content indexing issue #869
…-60_percent_watched
|
1 participant
No description provided.