Skip to content

Conversation

xitonix
Copy link
Contributor

@xitonix xitonix commented Sep 26, 2025

Make Message ID and Timestamp optional to support rETL, which cannot provide a Message ID. Without a Message ID, activities from rETL cannot be deduplicated using the usual key, so this change allows the integration to accept events while acknowledging that duplicates may occur.

Testing

  • Added unit tests for new functionality
  • Tested end-to-end using the local server
  • [If destination is already live] Tested for backward compatibility of destination. Note: New required fields are a breaking change.
  • [Segmenters] Tested in the staging environment
  • [Segmenters] [If applicable for this change] Tested for regression with Hadron.

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