Skip to content

fix: add timestamp setter#30

Merged
oliverb123 merged 1 commit intomainfrom
add-timestamp-setter
Jun 3, 2025
Merged

fix: add timestamp setter#30
oliverb123 merged 1 commit intomainfrom
add-timestamp-setter

Conversation

@oliverb123
Copy link
Copy Markdown
Contributor

This really ought to have always been available. We're more strict that the posthog pipeline here as regards dates being required to be in the past (there is a leeway of about a day in the ingestion pipeline, but that's due to noncompliant or buggy SDK implementations).

Copy link
Copy Markdown

@aviramha aviramha left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you

@oliverb123 oliverb123 merged commit 6657547 into main Jun 3, 2025
1 check 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.

2 participants