Skip to content

Update Intenze adapter: added new bidder params for region and placement - #4859

Open
devops473 wants to merge 2 commits into
prebid:masterfrom
devops473:update-intenze-adapter
Open

Update Intenze adapter: added new bidder params for region and placement#4859
devops473 wants to merge 2 commits into
prebid:masterfrom
devops473:update-intenze-adapter

Conversation

@devops473

Copy link
Copy Markdown

We want to integrate prebid server integration to our new ssp.
Added the new following bid params for our adapter:
region - optional target region of the world: us-east(default), eu, apac
placementId - optional, used when partner wants to integrate to our ssp
accountId - this old parameter is optional now and used when partner wants to integrate to our exchange.

@devops473

Copy link
Copy Markdown
Author

Hi, @bsardo and @przemkaczmarek, could you please review this merge pls? best regards

@przemkaczmarek przemkaczmarek self-assigned this Jul 21, 2026
@przemkaczmarek
przemkaczmarek self-requested a review July 21, 2026 12:16
@github-actions

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, 2ab4f13

intenze

Refer here for heat map coverage report

github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:37:	Builder				100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:49:	getHeaders			100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:72:	MakeRequests			81.8%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:100:	getImpressionExt		100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:117:	buildEndpointURL		50.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:159:	checkResponseStatusCodes	100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:169:	MakeBids			100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:214:	getMediaTypeForImp		100.0%
total:									(statements)			84.1%

Comment thread openrtb_ext/imp_intenze.go
Comment thread adapters/intenze/intenze.go Outdated
Comment thread adapters/intenze/intenze.go
Comment thread adapters/intenze/intenze.go Outdated
Comment thread adapters/intenze/intenze.go Outdated
Comment thread adapters/intenze/params_test.go
@devops473

Copy link
Copy Markdown
Author

@przemkaczmarek hey, thanks for your review, pushed fixes

@devops473

Copy link
Copy Markdown
Author

@przemkaczmarek hi! could you please check the fixes? thanks in advance!

@github-actions

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, 6bdc007

intenze

Refer here for heat map coverage report

github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:34:	Builder				100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:39:	getHeaders			100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:62:	MakeRequests			90.9%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:90:	getImpressionExt		100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:107:	buildEndpointURL		100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:146:	checkResponseStatusCodes	100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:156:	MakeBids			100.0%
github.com/prebid/prebid-server/v4/adapters/intenze/intenze.go:201:	getMediaTypeForImp		100.0%
total:									(statements)			98.8%

@devops473

Copy link
Copy Markdown
Author

@przemkaczmarek hi! thanks for your approve, are we ready to merge it?

@przemkaczmarek

Copy link
Copy Markdown
Collaborator

@devops473 Hi, we need 2nd approval, then we can merge it.

@devops473

devops473 commented Aug 1, 2026

Copy link
Copy Markdown
Author

@przemkaczmarek should we tag someone who can merge it or just wait? thanks

@przemkaczmarek

Copy link
Copy Markdown
Collaborator

@bsardo LGTM

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