Skip to content

Ogury: Allow inventory mapping for in-app traffic.#4636

Merged
bsardo merged 1 commit into
prebid:masterfrom
Ogury:ogury-ADV-28940-allow-inapp-mapping
Jan 28, 2026
Merged

Ogury: Allow inventory mapping for in-app traffic.#4636
bsardo merged 1 commit into
prebid:masterfrom
Ogury:ogury-ADV-28940-allow-inapp-mapping

Conversation

@crepi
Copy link
Copy Markdown
Contributor

@crepi crepi commented Dec 15, 2025

We no longer require assetKey and adUnitId for in-app traffic, it can be handled with app.publisher.id with inventory mapping (like it did for web).

Doc update: prebid/prebid.github.io#6371

We no longer require assetKey and adUnitId for in-app traffic,
it can be handled with app.publisher.id with inventory mapping
(like it did for web).
@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, 5d995fb

ogury

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/ogury/ogury.go:22:	Builder			100.0%
github.com/prebid/prebid-server/v3/adapters/ogury/ogury.go:26:	MakeRequests		76.3%
github.com/prebid/prebid-server/v3/adapters/ogury/ogury.go:117:	buildHeaders		50.0%
github.com/prebid/prebid-server/v3/adapters/ogury/ogury.go:130:	getMediaTypeForBid	33.3%
github.com/prebid/prebid-server/v3/adapters/ogury/ogury.go:147:	hasPublisherId		100.0%
github.com/prebid/prebid-server/v3/adapters/ogury/ogury.go:151:	hasSitePublisherId	100.0%
github.com/prebid/prebid-server/v3/adapters/ogury/ogury.go:155:	hasAppPublisherId	100.0%
github.com/prebid/prebid-server/v3/adapters/ogury/ogury.go:159:	MakeBids		70.0%
total:								(statements)		69.7%

Copy link
Copy Markdown

@Taxel Taxel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

4 participants