Skip to content
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

BlueSea: Add site capability #3910

Merged
merged 2 commits into from
Oct 16, 2024
Merged

Conversation

blueseasx
Copy link
Contributor

To include declartion of site capability in bidder-info YAML file and create site test cases.

@bsardo bsardo changed the title Add site capability BlueSea: Add site capability Sep 11, 2024
@blueseasx
Copy link
Contributor Author

Hi @bsardo thanks for your correction on the title, would like to konw if there are any other problems I should address. One of our partners is waiting for site integration.

@@ -3,9 +3,15 @@ maintainer:
email: [email protected]
endpointCompression: gzip
modifyingVastXmlAllowed: true
gvlVendorID: 1294
Copy link
Collaborator

@przemkaczmarek przemkaczmarek Sep 25, 2024

Choose a reason for hiding this comment

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

Verified:

"1294": {
      "id": 1294,
      "name": "Bluesea Holdings Hong Kong Limited",
      "purposes": [1],
      "legIntPurposes": [],
      "flexiblePurposes": [],
      "specialPurposes": [],
      "features": [],
      "specialFeatures": [],
      "cookieMaxAgeSeconds": null,
      "usesCookies": false,
      "cookieRefresh": false,
      "usesNonCookieAccess": true,
      "dataRetention": {
        "purposes": {

        },
        "specialPurposes": {

        }
      },
      "urls": [
        {
          "langId": "en",
          "privacy": "https://www.blueseasx.com/policy.html",
          "legIntClaim": "https://www.blueseasx.com/policy.html"
        }
      ],
      "dataDeclaration": [1, 2, 3, 4, 8, 11],
      "deviceStorageDisclosureUrl": "https://www.blueseasx.com/tcf-disclosure.json"
    },
    ```

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @bsardo do you have any further question regarding this PR? If possbile, could you help to approve it at your earliest convenience. My parnter has been waiting for over 5 weeks. Really appreciate your support.

Also thanks for review from @przemkaczmarek

Copy link
Collaborator

Choose a reason for hiding this comment

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

Sorry for the delay. LGTM. Merged.

@bsardo bsardo merged commit 451bc44 into prebid:master Oct 16, 2024
3 checks passed
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.

3 participants