-
Notifications
You must be signed in to change notification settings - Fork 160
feat: push replay and person data to intercom and crisp chat #2039
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
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
Adds integration support for Intercom and Crisp Chat to automatically link PostHog session replays and person data with chat conversations.
- New external integration system in
packages/browser/src/extensions/external-integration.ts
manages dynamic loading and lifecycle of chat integrations - Security concerns in
packages/browser/playground/nextjs/pages/external_chat.tsx
with hardcoded API keys/IDs and script loading approach - Extended type system in
packages/browser/src/types.ts
with newExternalIntegrationKind
and config options for integration management - Implementation in
packages/browser/src/entrypoints/intercom-integration.ts
andcrisp-chat-integration.ts
tracks session IDs to prevent duplicate updates - CSP updates in
packages/browser/playground/nextjs/pages/_app.tsx
to allow connections to third-party chat services
11 files reviewed, 8 comments
Edit PR Review Bot Settings | Greptile
Size Change: +10.7 kB (+0.3%) Total Size: 3.58 MB
ℹ️ View Unchanged
|
replaces #1980
easier to recreate than fangle the merge
if you've got intercom or crisp chat on your site, you might want to auto-link session replay to user interactions
you can do this manually but it might be nice to make it easy for folk
crisp
intercom