Skip to content

Conversation

tcgilbert
Copy link
Contributor

Update reddit capi destination name per partner request.

Testing

n/a

Copy link
Contributor

New required fields detected

Warning

Your PR adds new required fields to an existing destination. Adding new required settings/mappings for a destination already in production requires updating existing customer destination configuration. Ignore this warning if this PR is for a new destination with no active customers in production.

The following required fields were added in this PR:

  • Destination: Reddit Conversions API, Settings:ad_account_id,conversion_token
  • Destination: Reddit Conversions API, Action:standardEvent, Fields:event_at,tracking_type
  • Destination: Reddit Conversions API, Action:customEvent, Fields:event_at,custom_event_name

Add these new fields as optional instead and assume default values in perform or performBatch block.

@joe-ayoub-segment joe-ayoub-segment merged commit d3e3281 into main Dec 13, 2024
14 checks passed
@joe-ayoub-segment joe-ayoub-segment deleted the thomas/reddit branch December 13, 2024 16:26
@joe-ayoub-segment
Copy link
Contributor

hi @tcgilbert PR deployed.

julesbravo pushed a commit to Stodge-Inc/segment-destination that referenced this pull request Mar 28, 2025
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