feat: hubvisor adapter#1
Conversation
nlequain
force-pushed
the
feat/hubvisor-generic-adapter
branch
4 times, most recently
from
7b7590f to
6d80665
Compare
nlequain
force-pushed
the
feat/hubvisor-generic-adapter
branch
from
6d80665 to
929a32d
Compare
|
|
||
| const videoParametersByBidId = {}; | ||
|
|
||
| function isTest() { |
There was a problem hiding this comment.
Pas sur que ça soit nécessaire d'en faire une fonction vu que ça n'est utilisé qu'une fois
There was a problem hiding this comment.
En soit ça permet de simplifier le code qui l'utilise en enlevant la complexité de "comment l'information est stockée ?", et ça coute à peu près rien de garder une fonction en plus, donc ça me semble OK de garder cette forme.
There was a problem hiding this comment.
Je vais laisser en l'état pour l'instant pour livrer l'adapteur à Prisa rapidement. Je reviendrai sur tes commentaires plus tard avant de publier l'adapteur.
| window.HbvPlayer.playOutstream(container, options); | ||
| } | ||
|
|
||
| function getSelector(config, bid) { |
There was a problem hiding this comment.
nécessaire d'en faire une fonction ?
There was a problem hiding this comment.
Pareil ici, ça permet d'abstraire une partie de la complexité, donc à part si ça te choque, perso ça me semble pas mal comme ça.
There was a problem hiding this comment.
Je vais laisser en l'état pour l'instant pour livrer l'adapteur à Prisa rapidement. Je reviendrai sur tes commentaires plus tard avant de publier l'adapteur.
| }); | ||
| } | ||
|
|
||
| function playOutstream(containerOrSelector, options) { |
There was a problem hiding this comment.
nécessaire d'en faire une fonction ?
There was a problem hiding this comment.
Idem ici, ça permet de découper le code au lieu d'avoir une grosse fonction plus complexe, donc à part si ça te semble vraiment pas clair, sinon je suis chaud de garder ça comme ça.
There was a problem hiding this comment.
Je vais laisser en l'état pour l'instant pour livrer l'adapteur à Prisa rapidement. Je reviendrai sur tes commentaires plus tard avant de publier l'adapteur.
3 participants
Description
Implement a new
Hubvisor Bid adaptermodule that connects to Hubvisor demand sources and supports the banner and video media types.