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

Identify User at registration, rather than on return to app (backport #1600) #1601

Merged
merged 1 commit into from
Jan 20, 2023

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Jan 20, 2023

Description

Our PostHog data showing anonymous users filling out the form, and signing up. We then get new users appear when they login, having following the relevant link in their e-mail.

PostHog is struggling to map these sessions to the same person, for reasons I am trying to understand, but I have been assured from PostHog dev team:

This is our team’s highest priority right now, we plan on testing a new algorithm next week (on our own data), but it’s too early on to commit on an ETA

The only proposal I have at the moment, is that we call the $identify before they navigate away to interact with their email address, i.e. exactly at the point they register. This assures that they still have the same anon ID from their original Website session.

This is also meaning that the user is then classified as two PostHog Persons, the second of which has inaccurate referral data, because it believe they came directly to PostHog from the e-mail, the original referal data form their anon ID cookie is lost.

Related Issue(s)

Checklist

  • I have read the contribution guidelines
  • [-] Suitable unit/system level tests have been added and they pass
  • [-] Documentation has been updated
  • [-] Upgrade instructions
  • [-] Configuration details
  • [-] Concepts
  • [-] Changes flowforge.yml?
  • [-] Issue/PR raised on flowforge/helm to update ConfigMap Template
  • [-] Issue/PR raised on flowforge/CloudProject to update values for Staging/Production

Labels

  • Backport needed? -> add the backport label
  • [-] Includes a DB migration? -> add the area:migration label

@knolleary knolleary merged commit fe6a1bb into maintenance Jan 20, 2023
@knolleary knolleary deleted the backport-1600 branch January 20, 2023 11:30
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