-
Notifications
You must be signed in to change notification settings - Fork 44
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
[FINNA-1182] Add support for Icareus player #2768
Conversation
Laitoin @pasitiis jos sul o jotai sanottavaa mut eipä täs hirveesti oo tyylimuutoksia. |
@LuomaJuha Mitä jos kuitenkin olisi icareus-helper, jolle vaan välitetään pelkän id:n sijaan koko polku tai koko url? Muuten ei voi olla tätäkään. Ehkä url voisi olla samaan tapaan konffeissa source-kohtaisesti kuin poster_sources. |
@EreMaijala ah, totta en huomioinu tuota kaveria. Voisihan se olla näppärä. Muokkaan vähän, että tulis jotenkin järkevän oikea. |
@EreMaijala noni, tajusin et miehän voin vaa palauttaa suoraan sen linkin siitä recordlinkeristä ku ne embedit on niillä aina samoja. Tein iframeen icareus funktion taas niin nyt ne toimii kun laittaa url:n ja categoriat. |
Eikö niissä linkeissä ole joku 'elonet' mukana myös? Mistä se tulee? Vai eikö näytetä koskaan missään muualla kuin elonet.finna.fi:ssä? |
@EreMaijala joo, se ei oo vaan tos host:ssa mut tos pathissa ilmenee. Kyllähän noita videoita voi muutkin käyttää sitten, kun se alku on todennäköisesti aina sama. |
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.
Testailin embed ja popup -tilassa. Näytti toimivan.
Adds a handler for Icareus videos
Test: Record/kavi.elonet_elokuva_1292557?imgid=1
Different tests: popup, config.ini > embedvideo, record page video
And for content page:
php: $this->iframe()->icareus('256244280', ['video'])