-
Notifications
You must be signed in to change notification settings - Fork 744
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New Adapter: Loopme #4023
base: master
Are you sure you want to change the base?
New Adapter: Loopme #4023
Conversation
Code coverage summaryNote:
loopmeRefer here for heat map coverage report
|
Code coverage summaryNote:
loopmeRefer here for heat map coverage report
|
Hi @ambalashov, we recently released PBS 3.0, more specifically v3.1.0, which updates Prebid Server package import references throughout the project from
As a result, please merge with master and then ensure all Prebid Server package import references in the files you’ve changed are |
# Conflicts: # exchange/adapter_builders.go
Code coverage summaryNote:
loopmeRefer here for heat map coverage report
|
@PyjamaWarrior can you please review? |
@@ -0,0 +1,20 @@ | |||
endpoint: http://prebid.loopmertb.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Endpoint is reachable:
curl -i --location --request POST http://prebid.loopmertb.com
HTTP/1.1 204 No Content
date: Thu, 14 Nov 2024 08:44:47 GMT
- global | ||
maintainer: | ||
email: "[email protected]" | ||
gvlVendorID: 109 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Verified gvl id:
{
"id": 109,
"name": "LoopMe Limited",
"purposes": [
1,
2,
3,
4,
7,
9,
10
],
"legIntPurposes": [],
"flexiblePurposes": [],
"specialPurposes": [
1,
2,
3
],
"features": [
1,
2,
3
],
"specialFeatures": [
1,
2
],
"overflow": {
"httpGetLimit": 32
},
"cookieMaxAgeSeconds": 7776000,
"usesCookies": true,
"cookieRefresh": true,
"urls": [
{
"langId": "en",
"privacy": "https://legal.loopme.com/privacy-center#contract-hyartvn1o",
"legIntClaim": "https://legal.loopme.com/privacy-center#contract-hyartvn1o"
}
],
"usesNonCookieAccess": true,
"dataRetention": {
"stdRetention": 396,
"purposes": {},
"specialPurposes": {}
},
"dataDeclaration": [
1,
2,
3,
4,
6,
7,
8,
9,
10,
11
],
"deviceStorageDisclosureUrl": "https://co.loopme.com/deviceStorageDisclosure.json"
}
adapters/loopme/loopme.go
Outdated
bid := &sb.Bid[i] | ||
bidType, err := getBidType(bid) | ||
if err != nil { | ||
return nil, []error{err} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After the first error in the loop adapter returns one error and nil value. Maybe continuing and returning all errors at the end (like in MakeRequests) would probably be better so that any valid bids are processed? I would recommend this approach but it is ultimately up to you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sense. Thanks.
I have sent a verification email. Please respond
|
Code coverage summaryNote:
loopmeRefer here for heat map coverage report
|
We sent a response on Nov 21, 9:59 AM . Thanks. |
No description provided.