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

Brightcom adapter: remove adapters #5071

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

prBigBrother
Copy link
Contributor

🏷 Type of documentation

  • other

📋 Checklist

  • Related pull requests in prebid.js or server are linked -> Paste link in this list or reference it on the PR itself

I've initiated a pull request to remove two adapters, Brightcom and BrightcomSSP, in the next major release. This is in line with our ongoing rebranding efforts and related to the next pull request prebid/Prebid.js#10924

Copy link

netlify bot commented Jan 10, 2024

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit c311a59
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/659e65a6add0ef000832ca15
😎 Deploy Preview https://deploy-preview-5071--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 site configuration.

@muuki88
Copy link
Contributor

muuki88 commented Jan 11, 2024

@prBigBrother usually we keep old adapters in the docs and you can add a disclaimer for the version since when the adapter is not available anymore and what should be used instead.

This makes it possible that old users still find the docs. Ultimately it's your decision, if you want to leave as few traces as possible and remove it for example if the old adapter will stop working entirely.

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

2 participants