feat(cdp): add calendar-day deduplication options for destinations#55433
Merged
meikelmosby merged 6 commits intomasterfrom Apr 21, 2026
Merged
feat(cdp): add calendar-day deduplication options for destinations#55433meikelmosby merged 6 commits intomasterfrom
meikelmosby merged 6 commits intomasterfrom
Conversation
Adds two new trigger options for destinations: "Run once per person per calendar day" and "Run once per person per event name per calendar day". These use the date in the masking hash (via formatDateTime(now())) so the dedup boundary is the calendar day, not a rolling time window. This solves the case where a 24h rolling interval suppresses events across day boundaries (e.g. 3pm Monday event masks 10am Tuesday event). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Contributor
|
Size Change: +481 B (0%) Total Size: 129 MB ℹ️ View Unchanged
|
Contributor
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
Adds "Once per calendar day" frequency option to workflows (HogFlows), matching the destination feature. Parameterizes the calendar-day masker tests to reduce duplication and adds a HogFlow-specific calendar day test. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
dmarchuk
approved these changes
Apr 21, 2026
Contributor
dmarchuk
left a comment
dmarchuk
left a comment
There was a problem hiding this comment.
I think at least the banner change would be good to adjust before merging, but otherwise looks good!
When the "per event name per calendar day" option is selected, the info banner now suggests "Run once per person per calendar day" instead of "Run once per person per interval" as the alternative. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The calendar day boundary uses UTC. Person timezone support via ifNull(person.properties.$geoip_time_zone) causes bytecode evaluation issues in the masker and needs further investigation. For now, keep UTC and make it explicit in the option labels with "(UTC)" suffix. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds two new trigger options for destinations: "Run once per person per calendar day" and "Run once per person per event name per calendar day". These use the date in the masking hash (via formatDateTime(now())) so the dedup boundary is the calendar day, not a rolling time window.
This solves the case where a 24h rolling interval suppresses events across day boundaries (e.g. 3pm Monday event masks 10am Tuesday event).
closes https://posthog.com/questions/custom-deduplication-logic
Destinations
Workflows
