Skip to content

Raptive custom min version - #93

Open
eyvazahmadzada wants to merge 3 commits into
raptive_min_version_from_0.38from
raptive_custom_min_version
Open

Raptive custom min version#93
eyvazahmadzada wants to merge 3 commits into
raptive_min_version_from_0.38from
raptive_custom_min_version

Conversation

@eyvazahmadzada

Copy link
Copy Markdown
Collaborator

Type of change

  • Bugfix

  • Feature

  • New bidder adapter

  • Updated bidder adapter

  • Code style update (formatting, local variables)

  • Refactoring (no functional changes, no api changes)

  • Build related changes

  • CI related changes

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

  • Other

Description of change

Other information

@eyvazahmadzada eyvazahmadzada mentioned this pull request Sep 8, 2026
10 tasks
@@ -1,98 +0,0 @@
import { logError } from '../../src/utils.js';

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@eyvazahmadzada Why this was removed, don'y they use any kind of configuration based on browser? For examplebrowserBlackList

@eyvazahmadzada eyvazahmadzada Sep 9, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is their configuration:

{
    "partner": ...,
    "timeoutInMillis": 500,
    "partnerClientId": "...",
    "partnerClientIdType": 3,
    "manualWinReportEnabled": true,
    "group": "A",
    "ABTestingConfigurationSource": "group"
}

So, seems like nope

* `window.intentIqAnalyticsAdapter_<partnerId>.reportExternalWin()`.
* Defaults to `false`.
*/
manualWinReportEnabled?: boolean;

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

As I remember when they used Analytical adapter, they used manual reporting, pls ask Shahar about it

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

They do set manualWinReportEnabled, but it won't break anything because we will simply not use it (it's always true)

/**
* Geo-region routing hint for the reporting server.
*/
region?: string;

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Also I think they could be active in Europe as well, again pls clarify it with Shahar

* 3 = adUnitCode only
* 4 = placementId only
*/
adUnitConfig?: 1 | 2 | 3 | 4;

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Not sure about this as well

/**
* Publisher domain name appended to report URLs.
*/
domainName?: string;

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Don't they use it?

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.

3 participants