Skip to content

Agt 632 migrating modules to type script - #89

Merged
DimaIntentIQ merged 8 commits into
release_0.38from
AGT-632-migrating-modules-to-type-script
Aug 3, 2026
Merged

Agt 632 migrating modules to type script#89
DimaIntentIQ merged 8 commits into
release_0.38from
AGT-632-migrating-modules-to-type-script

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

Eyvaz Ahmadzada and others added 5 commits July 5, 2026 16:56
* 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>
@github-actions

Copy link
Copy Markdown

This PR introduces changes that may not work on all browsers. According to Babel, the following polyfills may be needed, and they are not automatically included:

  • Changes to libraries/intentIqUtils/detectBrowserUtils.ts may need:
    • es.iterator.constructor
    • es.iterator.map
    • es.iterator.some
    • esnext.iterator.constructor
    • esnext.iterator.map
    • esnext.iterator.some
  • Changes to libraries/intentIqUtils/gamPredictionReport.ts may need:
    • es.array.push
    • es.iterator.constructor
    • es.iterator.filter
    • es.iterator.find
    • es.iterator.for-each
    • esnext.iterator.constructor
    • esnext.iterator.filter
    • esnext.iterator.find
    • esnext.iterator.for-each
  • Changes to libraries/intentIqUtils/getRefferer.ts may need:
    • web.url-search-params.delete
    • web.url-search-params.has
    • web.url-search-params.size
  • Changes to libraries/intentIqUtils/storageUtils.ts may need:
    • es.iterator.constructor
    • es.iterator.filter
    • es.json.stringify
    • esnext.iterator.constructor
    • esnext.iterator.filter
    • esnext.json.parse
  • Changes to libraries/intentIqUtils/urlUtils.ts may need:
    • es.json.stringify
  • Changes to modules/intentIqAnalyticsAdapter.ts may need:
    • es.array.push
    • es.json.stringify
    • web.dom-exception.stack
  • Changes to modules/intentIqIdSystem.ts may need:
    • es.iterator.constructor
    • es.iterator.for-each
    • es.json.stringify
    • esnext.iterator.constructor
    • esnext.iterator.for-each

The best way to address this is to provide good test coverage, as normal PR checks run unit tests on older browsers.

@github-actions

github-actions Bot commented Aug 3, 2026

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.

export const EMPTY = "EMPTY";
export const GVLID = "1323";
export const GVLID = 1323;
export const VERSION = 0.371;

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.

version 0.38

@DimaIntentIQ
DimaIntentIQ merged commit 0ba2b4a into release_0.38 Aug 3, 2026
15 of 16 checks passed
@DimaIntentIQ
DimaIntentIQ deleted the AGT-632-migrating-modules-to-type-script branch August 3, 2026 14:58
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.

3 participants