Skip to content

Agt 632 migrating modules to type script#89

Open
eyvazahmadzada wants to merge 5 commits into
release_0.38from
AGT-632-migrating-modules-to-type-script
Open

Agt 632 migrating modules to type script#89
eyvazahmadzada wants to merge 5 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.

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