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

feat: Sentry FullStory integration #322

Merged
merged 4 commits into from
Apr 25, 2024
Merged

Conversation

cmaddox5
Copy link
Contributor

@cmaddox5 cmaddox5 commented Apr 24, 2024

Asana task: ad-hoc

This PR adds the FullStory integration to our Sentry initialization. Unfortunately, we do not track FullStory sessions in our dev environments and FullStory is configured to prevent session capturing for localhost so I'm not sure how to verify the integration works unless we deploy to prod. I can confirm that Sentry is still receiving events even if FullStory session URLs don't exist. https://mbtace.sentry.io/issues/5249154602/events/22a637e1840947da88428030d55913ba/.


@cmaddox5 cmaddox5 requested review from a team and panentheos and removed request for a team April 24, 2024 14:01
Copy link
Collaborator

@panentheos panentheos left a comment

Choose a reason for hiding this comment

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

Looks reasonable.

@cmaddox5 cmaddox5 merged commit 7e7f207 into main Apr 25, 2024
4 checks passed
@cmaddox5 cmaddox5 deleted the cm/sentry-fullstory-integration branch April 25, 2024 14:29
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