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

Pass user information to fullstory #9973

Merged
merged 2 commits into from
Feb 2, 2022
Merged

Conversation

timroes
Copy link
Collaborator

@timroes timroes commented Feb 1, 2022

What

Fixes #9747

Pass user information of the authenticated user to FullStory's identify method, to be able to filter for user in fullstory.

Strictly spoken since we pass through that method once before we have the information from the auth service, even when logged in, we call anonymize once also for logged in users. I validated how this looks in fullstory, and this is not causing any troubles.

@github-actions github-actions bot added the area/platform issues related to the platform label Feb 1, 2022
@timroes timroes temporarily deployed to more-secrets February 1, 2022 17:34 Inactive
Copy link
Contributor

@jamakase jamakase left a comment

Choose a reason for hiding this comment

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

Just a few small questions. Apart from that - LGTM.

@timroes timroes temporarily deployed to more-secrets February 1, 2022 20:22 Inactive
@timroes timroes temporarily deployed to more-secrets February 2, 2022 09:40 Inactive
@timroes timroes merged commit a6d89ce into master Feb 2, 2022
@timroes timroes deleted the tim/webapp/fullstory-identify branch February 2, 2022 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/platform issues related to the platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use logged in user information to identify in Fullstory
2 participants