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

ShowHeroes Bid Adapter: update params #3956

Merged
merged 6 commits into from
Aug 17, 2022
Merged

Conversation

vadim-mazzherin
Copy link
Contributor

🏷 Type of documentation

  • update bid adapter

📋 Checklist

@netlify
Copy link

netlify bot commented Aug 10, 2022

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit b126fa8
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/62f3ec59d3c3d3000895fd65
😎 Deploy Preview https://deploy-preview-3956--prebid-docs-preview.netlify.app/dev-docs/bidders/showheroes-bs
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@vadim-mazzherin
Copy link
Contributor Author

I'm not sure about the "required (if not send unitId)".
Maybe there is another option to specify that one of the params is required

@fowler446 fowler446 merged commit f18b25a into prebid:master Aug 17, 2022
ecdrsvc pushed a commit to ecdrsvc/prebid.github.io that referenced this pull request Sep 6, 2023
* add ShowHeroes Adapter

* removing longer-than-12 header

happily we don't need that anymore

* ShowHeroes Bid Adapter: update params

* Delete showheroes.md

Co-authored-by: bretg <bgorsline@gmail.com>
jlaso pushed a commit to AuDigent/prebid.github.io that referenced this pull request Nov 6, 2024
* add ShowHeroes Adapter

* removing longer-than-12 header

happily we don't need that anymore

* ShowHeroes Bid Adapter: update params

* Delete showheroes.md

Co-authored-by: bretg <bgorsline@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants