From 6bb211c4bd7d000a354aa9a4fdb2fc1c11872452 Mon Sep 17 00:00:00 2001 From: Isuru-F <61958023+Isuru-F@users.noreply.github.com> Date: Wed, 8 Jul 2020 15:23:30 +1000 Subject: [PATCH 1/2] Added clarification on video content position / total length --- src/connections/spec/video.md | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/connections/spec/video.md b/src/connections/spec/video.md index bc63f40b12..0e0eeaa969 100644 --- a/src/connections/spec/video.md +++ b/src/connections/spec/video.md @@ -73,7 +73,7 @@ The type of ad playing at the time of the event. Values can include 'pre-roll', #### Position: `Integer` -The current index position **in seconds** of the playhead. If the playback is a livestream, check the documentation for relevant destinations for details on how to correctly pass the playhead position. +The current index position **in seconds** of the playhead. If available - this should includes the duration of any ads seen. If the playback is a livestream, check the documentation for relevant destinations for details on how to correctly pass the playhead position. #### Seek Position: `Integer` @@ -618,10 +618,12 @@ The channel in which the video content is playing, ie. 'espn', 'my blog'. An [ISO 8601 Date String](https://en.wikipedia.org/wiki/ISO_8601) representing the original air date or published date. #### Position: `Integer` -The current index position **in seconds** of the playhead. If the playback is a livestream, check the documentation for relevant destinations for details on how to correctly pass the playhead position. +The current index position **in seconds** of the playhead into the content/asset. This position must exclude the duration of any ads played. + +If the playback is a livestream, check the documentation for relevant destinations for details on how to correctly pass the playhead position. #### Total Length: `Integer` -The total duration of the playback in seconds. This should include the duration of all your content and ad included in this playback session. For livestream playback, send `null`. +The total duration of the content/asset in seconds. This should exclude the duration of any ads included in the playback of this asset. For livestream playback, send `null`. #### Bitrate: `Integer` The current `kbps`. From 81323262a5d30a732f34f0085a1afc4abce90dfc Mon Sep 17 00:00:00 2001 From: Isuru-F <61958023+Isuru-F@users.noreply.github.com> Date: Thu, 9 Jul 2020 08:52:46 +1000 Subject: [PATCH 2/2] Update src/connections/spec/video.md Co-authored-by: LRubin --- src/connections/spec/video.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/connections/spec/video.md b/src/connections/spec/video.md index 0e0eeaa969..310b547a74 100644 --- a/src/connections/spec/video.md +++ b/src/connections/spec/video.md @@ -73,7 +73,7 @@ The type of ad playing at the time of the event. Values can include 'pre-roll', #### Position: `Integer` -The current index position **in seconds** of the playhead. If available - this should includes the duration of any ads seen. If the playback is a livestream, check the documentation for relevant destinations for details on how to correctly pass the playhead position. +The current index position **in seconds** of the playhead, including the duration of any ads seen (if available). If the playback is a livestream, check the documentation for relevant destinations for details on how to correctly pass the playhead position. #### Seek Position: `Integer`