-
Notifications
You must be signed in to change notification settings - Fork 5
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-11608): Provider - Expose default service Url #157
Conversation
src/k-provider/ovp/index.js
Outdated
|
||
declare var __VERSION__: string; | ||
declare var __NAME__: string; | ||
|
||
const NAME = __NAME__ + '-ovp'; | ||
const VERSION = __VERSION__; | ||
|
||
export {OVPProvider as Provider, NAME, VERSION, RequestBuilder, ResponseTypes}; | ||
export {OVPProvider as Provider, NAME, VERSION, RequestBuilder, ResponseTypes, OVPConfiguration}; |
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.
export {OVPProvider as Provider, NAME, VERSION, RequestBuilder, ResponseTypes, OVPConfiguration}; | |
export {OVPProvider as Provider, NAME, VERSION, RequestBuilder, ResponseTypes, OVPConfiguration as DefaultConfiguration}; |
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.
@RoyBregman Maybe OVPDefaultConfiguration ? to highlight that is relevant only for OVP ?
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.
not needed. the player is already typed. there is kaltura-ovp-player and kaltura-tv-player
5cae090
to
b798efe
Compare
Description of the Changes
Provider - Expose default service Url
solves FEC-11608
related pr:
kaltura/playkit-js-kaltura-cuepoints#14
CheckLists