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

SOVRN ad example is broken #4614

Closed
lannka opened this issue Aug 18, 2016 · 10 comments
Closed

SOVRN ad example is broken #4614

lannka opened this issue Aug 18, 2016 · 10 comments
Assignees

Comments

@lannka
Copy link
Contributor

lannka commented Aug 18, 2016

yay positive response sovrn.amp.handleResponse({
  "id" : "342521",
  "seatbid" : [ ]
})
sovrn_ads.js:1 error is TypeError: Cannot read property 'bid' of undefined(…)

To reproduce, check here.
@mcofran could you please take a look?

@lannka lannka added this to the Pending 3P Implementation milestone Aug 18, 2016
@mcofran
Copy link
Contributor

mcofran commented Aug 22, 2016

Thank you Hongfei. I am working on getting this request into the appropriate hands.

Thanks,

Martha Cofran | Director of Engineering, AdTech
720-201-2691 Cell

mcofran@sovrn.com mailto:mcofran@sovrn.com
sovrn.com http://sovrn.com/ | sovrn pubhub http://pubhub.sovrn.com/ | @sovrnholdings http://twitter.com/sovrnholdings

http://www.sovrn.com/onscroll

On Aug 18, 2016, at 4:28 PM, Hongfei Ding notifications@github.com wrote:

yay positive response sovrn.amp.handleResponse({
"id" : "342521",
"seatbid" : [ ]
})
sovrn_ads.js:1 error is TypeError: Cannot read property 'bid' of undefined(…)
To reproduce, check here http://amphtml-nightly.herokuapp.com/examples/ads.amp.html#sovrn.
@mcofran https://github.com/mcofran could you please take a look?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub #4614, or mute the thread https://github.com/notifications/unsubscribe-auth/AGjpFfgD7w34uf5dggmb2yi4toGGoTe-ks5qhNyFgaJpZM4Jn-p4.

@sovrnoss
Copy link

@lannka

Looks like that sovrn tag isn't returning responses.
Our team will create an account with a tag that always returns "test" creatives.

We will update this thread with the replacement tag once that's complete.

Thanks,
sovrn Support Team

@lannka
Copy link
Contributor Author

lannka commented Aug 22, 2016

@sovrnoss that will be awesome. thanks for looking into it.

@sovrnoss
Copy link

@lannka

Thank you for being patient with us on this, we have a created a tag that will return a test creative, the id is 393900. Please let us know if you need any additional information.

Thanks,
sovrn Support Team

@sovrnoss
Copy link

@lannka

Have you had a chance to evaluate our new ad tag? Can we close this issue or do you need additional information?

Thanks,
sovrn Support Team

@lannka
Copy link
Contributor Author

lannka commented Nov 29, 2016

sorry @sovrnoss I missed your previous reply.

unfortunately I still can't get it right with your new ID.

yay positive response sovrn.amp.handleResponse({
  "id" : "393900",
  "seatbid" : [ ]
})
sovrn_ads.js:1 error is TypeError: Cannot read property 'bid' of undefined(…)

@sovrnoss
Copy link

@lannka we've tested those zones on our end and they have been working, we're wondering what domain you ran your test on and if it needs to be whitelisted on our end. We whitelisted the http://amphtml-nightly.herokuapp.com/ domain, but any others will not receive a response from us.

Thanks!
sovrn Support Team

@lannka
Copy link
Contributor Author

lannka commented Nov 29, 2016

@sovrnoss would it be possible for you to whitelist localhost:8000, for sake of manual test?

@sovrnoss
Copy link

@lannka localhost:8000 is now whitelisted and should be returning positive responses. Please confirm.

Thanks,
sovrn Support Team

@lannka
Copy link
Contributor Author

lannka commented Nov 29, 2016

@sovrnoss it works!
Thanks!

@lannka lannka self-assigned this Nov 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants