react-dfp
react-dfp copied to clipboard
jaanauati
Add amazon ATM support
This PR adds amazon transparent market place support (https://aps.amazon.com/aps/transparent-ad-marketplace/)
@evgenizhuk @jaanauati what are you thoughts on keeping this component brand agnostic of add-ons? We use Amazon TAM + react-dfp as well but simply utilize the built-in AdSlot props to connect it all and it works fine.
Just bringing this up because there are other bidding systems out there and if we go down the route of internally supporting other systems this project could balloon in scope & size. Open to discussion here.
Ps- here's the original PR that allowed react-dfp to work with Amazon TAM
https://github.com/jaanauati/react-dfp/pull/158
Ps- here's the original PR that allowed react-dfp to work with Amazon TAM
#158
@scott-thrillist thank you for the reference, I've missed this prop. Probably, it makes sense to update documentation.
@scott-thrillist could you provide an example? As I understand for amazon TAM enabling DFPSlotsProvider:autoReload:adUnit should be set to true and register bids fetching as onSlotRegister callback. Does a DFP single request architecture mode work correctly with it? Thank you
@yahenizhuk Here's a snippet I created for someone else. I may clean this up and make an example within this project when some time opens up.
https://gist.github.com/scott-thrillist/bf7d429f6c78a417fe4717a2e1791dc1
A potential downside here is we're not making a single request to Amazon but ideally you should be lazy loading ads to increase your viewability metrics. This would make new, separate requests as you scroll down the page vs. a single bigger payload and longer wait times during load.
@scott-thrillist Thank you for the example. Unfortunately, it doesn't fit our requirements. I'll try to approach it with a plugin.
@yahenizhuk Gotcha and thank you. Can I ask what are the requirements that this doesn't meet? Always interested in understanding how other organizations implement ad bidding with DFP.
Hi @scott-thrillist , thank you for quick answers. Unfortunately, I don't think I can share a lot due to our policy. I can say only that SRA mode is critically important for us also we need to add bidding, not to all ad units on the page so with disableInitialLoad in provider and refreshes in adSlot it's hard to control ads behavior.
But I like your idea about the plugin. I will continue with this and will return with the PR when it will be done. I think this PR can be closed. Thank you!
Thanks for sharing. We implement conditional ad bidding as well, and simply refresh the ad sans-TAM if it doesn't meet the condition. But I understand everyone has different requirements. Happy coding