Repository navigation
Mark modules and features removed in Prebid.js 11.0 - #6830
Conversation
Mark module, bidder, userId and publisher API pages for modules removed in Prebid.js 11.0 (adpod, categoryTranslation, dfpAdpod, freeWheelAdserverVideo, express, intersectionRtdProvider, paapi, paapiForGpt, topLevelPaapi, dmdIdSystem, optable and ringieraxelspringer bid adapters) as removed, and add removal notes to long-form video and PAAPI guides. Remove the stale ad pod how-to from the bidder adapter guide. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
✅ Deploy Preview for prebid-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 19b327367d
ℹ️ 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".
| --- | ||
| layout: api_prebidjs | ||
| title: pbjs.getPAAPIConfig(options) | ||
| title: pbjs.getPAAPIConfig(options) <span style="color:red" markdown="1">[Removed in 11.0]</span> |
There was a problem hiding this comment.
When a user opens this page, the new <span> is not rendered as a badge in the browser title: _includes/head--common.html:15 interpolates page.title directly inside <title>, where nested HTML is treated as literal text. This page and the other previously plain API titles changed in this patch will therefore expose the raw span markup in tab, bookmark, and search titles; set a plain head_title or render the badge outside the title value.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Fixed in 1499d0e: each of the five API pages with a badge in title now sets a plain head_title (e.g. pbjs.getPAAPIConfig(options) [Removed in 11.0] for Header Bidding), so the browser title has no markup. The badge stays in title for the API reference listing, matching the existing [Alpha] convention.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…edback Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
# Conflicts: # dev-docs/bidder-adaptor.md
|
@patmmccann can you kick claude to fix the merge conflicts 🙈 |
Prebid.js 11.0 removed adpod, categoryTranslation, dfpAdpod/gamAdpod, freeWheelAdserverVideo, express, intersectionRtdProvider, paapi, paapiForGpt, topLevelPaapi, dmdIdSystem, and the optable and ringieraxelspringer bid adapters (see the Prebid 11 notes). Many of their docs pages still presented them as available, and most were still offered on the download page.
This PR:
enable_download: falseandpbjs_version_notes: removed in 11.0on the affected module, bidder and userId pages, so they drop off the download page and module lists, and adds a "removed in Prebid.js 11.0" note to each.adServers.freewheel.getTargeting,adServers.gam.buildAdpodVideoUrl,getPAAPIConfig,getPAAPIBidsandsetPAAPIConfigForGPTAPI pages as removed in 11.0.adpodfrom the list of video contexts. The heading is kept so existing anchors still work.Pages are kept rather than deleted so publishers on older versions can still use them. Prebid Server docs and historical release notes are unchanged.
🤖 Generated with Claude Code