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

Concert Adapter: New Server Adapter #5293

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Conversation

BrettBlox
Copy link
Contributor

@BrettBlox BrettBlox commented May 1, 2024

Updates Concert Adapter docs in conjunction with our new Prebid Server adapter.
Prebid server adapter PR

🏷 Type of documentation

  • new bid adapter
  • update bid adapter

📋 Checklist

  • Related pull requests in prebid.js or server are linked -> Paste link in this list or reference it on the PR itself
  • For new adapters check submitting your adapter docs

Copy link

netlify bot commented May 1, 2024

Deploy Preview for prebid-docs-preview failed. Why did it fail? →

Name Link
🔨 Latest commit 9fab077
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/665de8e18820fb0008e4aab9

@ChrisHuie ChrisHuie added the LGTM label May 2, 2024
tcfeu_supported: true
usp_supported: true
gvl_id: N/A
coppa_supported: false
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add some additional documentation. This is below a reasonable minimum

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Skitelman tagging you here for visibility.

Copy link
Collaborator

@patmmccann patmmccann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

insufficient detail

@Skitelman
Copy link
Contributor

insufficient detail

Added more documentation in this commit. Let me know if that is sufficient. Thank you.

@BrettBlox BrettBlox requested a review from patmmccann June 3, 2024 21:52
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.

None yet

4 participants