Skip to content

'Module Change' Optable RTD module: remove unneeded logique. #13114

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

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

Conversation

jmarc101
Copy link

Type of change

  • Updated bidder adapter
  • Does this change affect user-facing APIs or examples documented on http://prebid.org?

Description of change

This PR refactors the Optable RTD provider module to simplify its implementation and improve maintainability. Key changes include:

  1. Removed the bundle URL loading mechanism in favor of requiring publishers to load the Optable SDK directly
  2. Simplified the configuration by removing bundleUrl and handleRtd parameters
  3. Added support for multiple Optable SDK instances through the new instanceName parameter
  4. Reduced recommended auction delay from 1000ms to 400ms
  5. Removed custom RTD handler functionality in favor of a simpler, more direct implementation
  6. Updated documentation to reflect these changes

The module now expects the Optable SDK to be loaded directly on the page and uses a simpler configuration model that focuses on the core functionality of enriching bid requests with targeting data.

Other information

  • This change affects user-facing APIs as documented in the module's documentation
  • The changes are backward compatible for existing implementations that use the default instance name
  • Documentation has been updated to reflect the new configuration options and requirements
  • No changes to the core functionality of enriching bid requests with Optable targeting data

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant