Skip to content

Conversation

@bTimor
Copy link
Contributor

@bTimor bTimor commented Sep 1, 2025

🏷 Type of documentation

  • new bid adapter

📋 Checklist

  • Related pull requests in prebid.js or server are linked -> The Prebid.js adapter code will be submitted separately
  • For new adapters check submitting your adapter docs

Description

This PR adds comprehensive documentation for the Scalibur bidder adapter, including:

  • Bidder Code: scalibur

  • GVLID: 1471

  • Media Types: Banner and Video (instream)

  • Privacy Compliance: Full support for GDPR (TCF EU), CCPA (USP), COPPA, and GPP

  • Key Features:

    • OpenRTB 2.x compliant requests
    • Supply chain transparency (schain)
    • Price floors support
    • First Party Data (FPD) support
    • User sync capabilities (iframe and image)
    • Multi-format ad unit support

    Related Pull Requests

  • Adapter Code PR: prebid/Prebid.js#13826 (will be submitted separately)

The documentation includes detailed configuration examples for banner, video, and multi-format ad units, along with comprehensive setup instructions for publishers.

Timor Bibi added 7 commits August 31, 2025 15:10
- Add complete bidder documentation with all required front matter
- Configure support for GDPR, CCPA, GPP, floors, and FPD
- Set GVLID to 1471 for proper GDPR compliance
- Enable integration with bidders list and search functionality
@netlify
Copy link

netlify bot commented Sep 1, 2025

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 1458443
🔍 Latest deploy log https://app.netlify.com/projects/prebid-docs-preview/deploys/6908664c6b0ba60008cb816c
😎 Deploy Preview https://deploy-preview-6235--prebid-docs-preview.netlify.app
📱 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 project configuration.

@lpagnypubstack
Copy link
Collaborator

@bTimor : can you please fix the rendering of your "Bid params" array within your file please : https://deploy-preview-6235--prebid-docs-preview.netlify.app/dev-docs/bidders/scalibur ?

@bTimor
Copy link
Contributor Author

bTimor commented Sep 2, 2025

@bTimor : can you please fix the rendering of your "Bid params" array within your file please : https://deploy-preview-6235--prebid-docs-preview.netlify.app/dev-docs/bidders/scalibur ?

Fixed. Thank you

@bTimor
Copy link
Contributor Author

bTimor commented Oct 12, 2025

@lpagnypubstack My adapter PR was approved and merged to master.
Adapter Code PR: prebid/Prebid.js#13826
The bid params rendering fix has been implemented and the code is ready for re-review.

@bTimor
Copy link
Contributor Author

bTimor commented Oct 23, 2025

Hi @patmmccann @muuki88,

I noticed you've been actively reviewing recent bidder documentation PRs. My Scalibur adapter documentation has been waiting for review for several weeks after I addressed the "needs work" feedback.

The Scalibur bidder adapter was already merged into Prebid.js, and this documentation PR completes the integration. Could you take a look when you have a moment?

PR has been kept up-to-date with master and all previous feedback has been addressed.

Thanks for your time!

@muuki88 muuki88 added LGTM and removed needs work labels Oct 23, 2025
Copy link
Contributor

@muuki88 muuki88 left a comment

Choose a reason for hiding this comment

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

Hi there,

Thanks for your patience. With Prebid Summit all volunteers were busy.

If you AI generate the documentation, please review yourself first and remove all redundancies yourself. This speeds up the review process and improves the documentation quality. Also use other bidder documentations in your context, so headings and styling are aligned.

@muuki88 muuki88 added needs work and removed LGTM labels Oct 23, 2025
@bTimor
Copy link
Contributor Author

bTimor commented Oct 27, 2025

@muuki88
I have fixed all your comments including other things that I compered with other bidder documentations.
Thanks for all the comments.

@bTimor bTimor requested a review from muuki88 October 27, 2025 09:06
biddercode: scalibur
gvl_id: 1471
media_types: banner, video
gpp_supported: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
gpp_supported: true

Copy link
Contributor

Choose a reason for hiding this comment

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

Would have been nice to see this removed. Can you do this in a small PR?

@muuki88 muuki88 merged commit 43b9e76 into prebid:master Nov 7, 2025
6 checks passed
@bTimor
Copy link
Contributor Author

bTimor commented Nov 9, 2025

@muuki88 Thanks for your review and merge.
I have created new small pr to remove the gpp_supported flag as requested.
you can see it here: New Scalibur PR

@bTimor bTimor deleted the scalibur branch November 9, 2025 10:58
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.

3 participants