Skip to content

Conversation

robbie-c
Copy link
Contributor

@robbie-c robbie-c commented Nov 12, 2024

Proposed changes

In posthog integration docs, add information about missing session IDs and how to fix

Merge timing

  • ASAP once approved

Related issues (optional)

Long term it would make sense for posthog to have a device mode integration (see PostHog/posthog#26153), but in the meantime it made sense to add this info to the docs

@robbie-c robbie-c requested a review from a team as a code owner November 12, 2024 18:56
@robbie-c robbie-c requested a review from pwseg November 12, 2024 18:56
@seg-atlantis-prod
Copy link

Atlantis commands can't be run on fork pull requests. To enable, set --allow-fork-prs or, to disable this message, set --silence-fork-pr-errors

@seg-atlantis-prod
Copy link

Error parsing command: EOF found when expecting closing quote

@robbie-c robbie-c changed the title Add requirements about session IDs to segment docs Add requirements about session IDs to posthog integration docs Nov 12, 2024
@robbie-c robbie-c changed the title Add requirements about session IDs to posthog integration docs Add info about session IDs to posthog integration docs Nov 12, 2024
@pwseg pwseg changed the title Add info about session IDs to posthog integration docs Add info about custom sessions IDs for the PostHog destination Jan 23, 2025
@pwseg pwseg added the content-update updates to content that are not new features, includes grammar fixes, added notes label Jan 23, 2025
@pwseg pwseg merged commit c3a12ba into segmentio:develop Jan 23, 2025
4 checks passed
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content-update updates to content that are not new features, includes grammar fixes, added notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants