-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat(FEC-11761): expose stream timed metadata - phase 2 #52
Conversation
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.
let's remove TODO that become outdated after your changes.
Except it PR LGTM
src/image-sync-manager.ts
Outdated
@@ -64,16 +53,16 @@ export class ImageSyncManager { | |||
this._imagePlayer.preLoadImages(); | |||
}; | |||
|
|||
private _onTimedMetadata = () => { | |||
private _onTimedMetadataChange = ({payload}: TimedMetadata) => { | |||
// TODO: use single "metadata" TextTrack once cue-point manager become use it |
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.
let's remove the TODO cause active cue-points already comes from payload of TIMED_METADATA_CHANGE
event
use `TimedMetadata` from core Change cuePointManager API's to use `TimedMetadata` instead of `VttCue` Solves FEC-11761 Depends on kaltura/playkit-js#623 Breaking Change for - kaltura/playkit-js-dual-screen#52 kaltura/playkit-js-kaltura-cuepoints#15
use
CuePoint
fromTIMED_METADATA_CHANGE
instead ofVttCue
fromTIMED_METADATA
Depends on kaltura/kaltura-player-js#512
Deleted and pushed by #53