(BE) Support VIDEO_60_PERCENT_WATCHED event type #855#423
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.
| * logged as normal. | ||
| * @throws SegueDatabaseException if the deduplication lookup fails. | ||
| */ | ||
| private Response handleVideoEngagementEvent(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.
c57e609 to
4b99327
Compare
Adds the VIDEO_60_PERCENT_WATCHED client log event: /log validation (logged-in user, payload checks, watch threshold) with per-user/per-video deduplication
4b99327 to
ebe4575
Compare
|
2 participants
No description provided.