Skip to content

New Adapter: Alliance Gravity#4502

Open
przemkaczmarek wants to merge 1 commit into
masterfrom
New-Adapter-Alliance-Gravity
Open

New Adapter: Alliance Gravity#4502
przemkaczmarek wants to merge 1 commit into
masterfrom
New-Adapter-Alliance-Gravity

Conversation

@przemkaczmarek
Copy link
Copy Markdown
Collaborator

🔧 Type of changes

  • new bid adapter

✨ What's the context?

#4403


final BidRequest modifiedRequest = request.toBuilder().imp(modifiedImps).build();
return Result.of(
Collections.singletonList(BidderUtil.defaultRequest(modifiedRequest, endpointUrl, mapper)),
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Add custom headers similar to GO

Comment on lines +118 to +121
if (bidType == null) {
return null;
}
return BidderBid.of(bid, bidType, currency);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

use ternary operator (? :)

}

private static BidType getBidType(Bid bid, List<BidderError> errors) {
final Integer mtype = bid.getMtype();
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

read from bid.ext.prebid.type

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

declare OpenRTB 2.6

- native
supported-vendors:
vendor-id: 501
usersync:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

add usersync.macro

usersync:
cookie-family-name: alliancegravity
iframe:
url: https://pbs.production.agrvt.com/static/cookie_sync.html?gdpr={{gdpr}}&gdpr_consent={{gdpr_consent}}&us_privacy={{us_privacy}}&redirect_url={{redirect_url}}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

redirect_url -> redirect

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants