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

Rubicon Adapter: Add CCPA and schain support #170

Merged
merged 9 commits into from
Sep 2, 2020

Conversation

idettman
Copy link

Type of Change

  • Feature Update

Description of Change

Add support for CCPA and schain

@ix-certification
Copy link
Contributor

ix-certification commented Feb 24, 2020

There are some issues with the PR submitted. Feedback has been posted on the HTP JIRA ticket but our feedback has not been addressed yet by Rubicon. This ticket is blocked.

Copy link

@harpere harpere left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@harpere harpere left a comment

Choose a reason for hiding this comment

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

LGTM

@ix-certification ix-certification merged commit 82a1a9e into indexexchange:canary Sep 2, 2020
@cd0n0van
Copy link

Hi @harpere and @idettman - we've observed an issue with the CCPA update in canary testing. We are seeing the following values returned: status: "error", reason: "internal-error", error_code: "18". Can you please confirm if this is related to the update?

@cd0n0van
Copy link

cd0n0van commented Nov 5, 2020

Hi @idettman and @harpere - wanted to check in on this update before our deployment freeze. Would you like IX to proceed with promoting this change to our master branch? Feel free to DM me with any questions or concerns.

@cd0n0van
Copy link

Hi @idettman & @harpere - IX currently still has this code in our canary branch. Would you like us to go ahead and promote the change, observing the aforementioned errors, or are you looking to submit a new PR? Let me know if there's any additional information IX can provide to aid in troubleshooting!

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.

4 participants