Skip to content

no ab group in browserblacklist - #87

Merged
DimaIntentIQ merged 6 commits into
release_0.38from
AGT-973-prebid-a-b-group-in-browser-black-list-mode
Jul 13, 2026
Merged

no ab group in browserblacklist#87
DimaIntentIQ merged 6 commits into
release_0.38from
AGT-973-prebid-a-b-group-in-browser-black-list-mode

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

Comment thread modules/intentIqIdSystem.ts Outdated

if (gamObjectReference) setGamReporting(gamObjectReference, gamParameterName, actualABGroup);
if (groupChanged) groupChanged(actualABGroup, partnerData?.terminationCause);
if (!isBlacklisted) {

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.

Nope, we shoudn't reach this part of code if it's listed, we call SYNC request instead of VR

@github-actions

Copy link
Copy Markdown

This PR includes an adapter whose code does not match its file name. Bid adapter modules should be named <bidderCode>BidAdapter, userId <userIdCode>IdSystem, RTD <rtdCode>RtdProvider, and analytics <analyticsCode>AnalyticsAdapter.

@DimaIntentIQ
DimaIntentIQ merged commit 566d48d into release_0.38 Jul 13, 2026
94 of 96 checks passed
@DimaIntentIQ
DimaIntentIQ deleted the AGT-973-prebid-a-b-group-in-browser-black-list-mode branch July 13, 2026 13:54
DimaIntentIQ pushed a commit that referenced this pull request Aug 3, 2026
* convert all iiq js files to ts

* update version, ts fixes

* fix test and lint issues

* fix lint issues

* no ab group in browserblacklist (#87)

* no ab group in browserblacklist

* fix lint issues

* remove isblacklisted logic from request

* small fix

* add tests

---------

Co-authored-by: Eyvaz Ahmadzada <eyvazahmadzada@gmail.com>

* small fixes

* update version

---------

Co-authored-by: Eyvaz Ahmadzada <eyvazahmadzada@gmail.com>
Co-authored-by: Eyvaz <eyvaz@intentiq.com>
DimaIntentIQ pushed a commit that referenced this pull request Aug 10, 2026
* convert all iiq js files to ts

* update version, ts fixes

* fix test and lint issues

* fix lint issues

* no ab group in browserblacklist (#87)

* no ab group in browserblacklist

* fix lint issues

* remove isblacklisted logic from request

* small fix

* add tests

---------



* small fixes

* update version

---------

Co-authored-by: Eyvaz Ahmadzada <eyvazahmadzada@gmail.com>
Co-authored-by: Eyvaz <eyvaz@intentiq.com>
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.

2 participants