-
Notifications
You must be signed in to change notification settings - Fork 2.2k
videoNow bid adapter #4088
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
videoNow bid adapter #4088
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.
Hi @sdbaron,
I am getting 404 Not Found
error when hitting the url https://bidder.videonow.ru/prebid?profile_id=1
. I am making request from India
with the following request payload: {"id":"2b9273013ea69d","sizes":[[640,480],[336,280]],"cur":"RUB","placementId":"div-001","ref":"http://localhost:9999/integrationExamples/gpt/hello_world.html?pbjs_debug=true"}
Please check at your end.
Hi! |
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.
Please make the suggested changes. I found unit test coverage is very low (~41%). Can you add more tests to increase it to at least 80%?
modules/videoNowBidAdapter.js
Outdated
const d = window.document | ||
const el = placementId && d.getElementById(placementId) | ||
if (!el) { | ||
console.error(`bidAdapter ${BIDDER_CODE}: ${placementId} not found`) |
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.
can we use utils.logError()
here?
modules/videoNowBidAdapter.js
Outdated
|
||
registerBidder(spec); | ||
|
||
function reInitScripts(container, shouldReInitInlineScripts = true, delay = 10) { |
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.
Can you please explain what does this function do and why do we need it in the adapter?
@@ -0,0 +1,203 @@ | |||
import { expect } from 'chai'; |
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.
unit test coverage is very low.
Hi @sdbaron, could you work on the above feedback or do you have any questions? |
Hi @sumit116, I had a vacation for a week, but now I’ve returned to work and will make corrections in a couple of days. |
Type of change
Description of change
new bidder adapter for videoNow