Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update snippet to include onSessionId #18484

Merged
merged 2 commits into from
Nov 8, 2023
Merged

Conversation

pauldambra
Copy link
Member

@pauldambra pauldambra commented Nov 8, 2023

We added a public method to posthog.js - I think that means we need to add it to the snippet

see: https://posthog.slack.com/archives/C0113360FFV/p1698322776615349

@pauldambra pauldambra requested a review from neilkakkar November 8, 2023 12:29
@posthog-bot
Copy link
Contributor

Hey @pauldambra! 👋
This pull request seems to contain no description. Please add useful context, rationale, and/or any other information that will help make sense of this change now and in the distant Mars-based future.

@pauldambra pauldambra merged commit aceec12 into master Nov 8, 2023
73 checks passed
@pauldambra pauldambra deleted the fix/add-on-session-id branch November 8, 2023 15:22
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.

3 participants