Skip to content

New Adapter: AdSmartX#4713

Closed
pritishmd-talentica wants to merge 5 commits into
prebid:masterfrom
smart-exchange-ai-digital:master
Closed

New Adapter: AdSmartX#4713
pritishmd-talentica wants to merge 5 commits into
prebid:masterfrom
smart-exchange-ai-digital:master

Conversation

@pritishmd-talentica

Copy link
Copy Markdown

Type of Change

New Bidder Adapter

Description of change

Added a bidder adapter for AdSmartX
Maintainer : prebid@aidigital.com

Bidder Params Documentation

prebid/prebid.github.io#6467

Comment thread adapters/adsmartx/adsmartx.go
Comment thread adapters/adsmartx/adsmartx.go
@github-actions

github-actions Bot commented Mar 6, 2026

Copy link
Copy Markdown

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, cfad8a6

adsmartx

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/adsmartx/adsmartx.go:19:	Builder		100.0%
github.com/prebid/prebid-server/v3/adapters/adsmartx/adsmartx.go:23:	MakeRequests	96.4%
github.com/prebid/prebid-server/v3/adapters/adsmartx/adsmartx.go:80:	parseImpExt	100.0%
github.com/prebid/prebid-server/v3/adapters/adsmartx/adsmartx.go:92:	MakeBids	100.0%
github.com/prebid/prebid-server/v3/adapters/adsmartx/adsmartx.go:128:	getBidType	100.0%
total:									(statements)	98.3%

Comment thread adapters/adsmartx/adsmartx.go
Comment thread adapters/adsmartx/adsmartx.go
@bsardo bsardo added the adapter label Mar 31, 2026
@pritishmd-talentica

Copy link
Copy Markdown
Author

Hi, Can we please have a review of this PR?

@pritishmd-talentica

Copy link
Copy Markdown
Author

Hi, Please can we have a review of this PR

@bsardo

bsardo commented Apr 23, 2026

Copy link
Copy Markdown
Collaborator

Hi @pritishmd-talentica, your adapter is importing v3 packages which are no longer valid. Please update your package references to v4. This is why some of the checks are failing.

@danylo-kazymyrov-teqblaze

Copy link
Copy Markdown

@pritishmd-talentica pls update the PR to pass CI checks

@pritishmd-talentica

Copy link
Copy Markdown
Author

Hi @bsardo & @danylo-kazymyrov-teqblaze , we are revamping this adapter and making some more changes. We can raise a fresh PR and close this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants