Skip to content

Remove pbadslot and customPbAdSlot from GPT Pre-Auction docs - #6612

Merged
muuki88 merged 2 commits into
masterfrom
codex/fix-issue-6435-on-github
Jul 27, 2026
Merged

Remove pbadslot and customPbAdSlot from GPT Pre-Auction docs#6612
muuki88 merged 2 commits into
masterfrom
codex/fix-issue-6435-on-github

Conversation

@patmmccann

@patmmccann patmmccann commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Fixes #6435

Motivation

  • Update the GPT Pre-Auction documentation to reflect that pbadslot / customPbAdSlot support has been removed from Prebid.js and the module now focuses on GPID population.
  • Reduce confusion by aligning the docs, examples, and config guidance with the current module behavior and API.

Description

  • Removed all references to pbadslot and the customPbAdSlot example and replaced them with GPID-focused wording and guidance.
  • Updated the configuration table and example to describe useDefaultPreAuction (defaulting to true) and the customPreAuction(adUnit, adServerAdSlot, gptAdUnitPath) signature.
  • Rewrote the “Defining GPID” flow to describe GPID-only population behavior and replaced the example with a customPreAuction sample using gptAdUnitPath.
  • Updated links to point to the GPID anchor (/features/pbAdSlot.html#the-gpid).

Testing

  • Installed markdownlint-cli via npm and ran markdownlint --config .markdownlint.json dev-docs/modules/gpt-pre-auction.md, and linting passed for the edited file.
  • Ran bundle install to ensure Jekyll dependencies were available and then ran bundle exec jekyll build, and the site built successfully without errors for the modified content.
  • No markdown lint or Jekyll build errors remain for the changed file.

Codex Task

@netlify

netlify Bot commented Jun 10, 2026

Copy link
Copy Markdown

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 183bc5e
🔍 Latest deploy log https://app.netlify.com/projects/prebid-docs-preview/deploys/6a677d0559607700081c3dab
😎 Deploy Preview https://deploy-preview-6612--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.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 047818b49d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread dev-docs/modules/gpt-pre-auction.md Outdated
Clarify the behavior of customPreAuction and default logic in GPID definition.
@patmmccann

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Another round soon, please!

Reviewed commit: 183bc5e4fa

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@patmmccann
patmmccann requested a review from ChrisHuie July 27, 2026 16:04
@patmmccann

Copy link
Copy Markdown
Collaborator Author

@ChrisHuie tagging you as you mentioned in earlier meeting this was ready to merge

@patmmccann
patmmccann requested review from dgirardi and muuki88 July 27, 2026 16:26
@muuki88
muuki88 merged commit 5740573 into master Jul 27, 2026
6 checks passed
@muuki88
muuki88 deleted the codex/fix-issue-6435-on-github branch July 27, 2026 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix pbadslot discussion

3 participants