Skip to content
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

Yieldmo adapter not setting creativeId #3054

Closed
beth92 opened this issue Sep 5, 2018 · 2 comments
Closed

Yieldmo adapter not setting creativeId #3054

beth92 opened this issue Sep 5, 2018 · 2 comments

Comments

@beth92
Copy link
Contributor

beth92 commented Sep 5, 2018

Type of issue

bug

Description

In the interpretResponse function for this adapter, a createNewBid function is invoked which sets creativeId to response.creativeId. However, the response from the server has snake case, not camel case, so should be reading response.creative_id.

https://github.com/prebid/Prebid.js/blob/master/modules/yieldmoBidAdapter.js#L114

Steps to reproduce

Set pbjs_debug=true query parameter. Emulate smartphone and send requests via prebid to yieldmo. Observe network requests panel to see response from yieldmo server. When a bid comes back (i.e. body is non-empty) check pbjs logs or call pbjs.getBidResponses() to observe that prebid records no yieldmo bid even though one was sent.

Platform details

Prebid.js version 1.19
Chrome 66
(emulating user agent: Mozilla/5.0 (Linux; Android 5.0; SM-G900P Build/LRX21T) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.117 Mobile Safari/537.36)

Other information

Created PR with proposed fix: #3053

@bretg
Copy link
Collaborator

bretg commented Sep 6, 2018

Merged #3053

@bretg bretg closed this as completed Sep 6, 2018
@harquail
Copy link

Thank you very much @beth92 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants