-
Notifications
You must be signed in to change notification settings - Fork 34
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
swapping posthog-js and posthog-js-lite #183
Comments
@pauldambra, hi! I only use |
hey @fromaline yep, we use it in the toolbar... so it should be completely fine this issue was only about the mismatch in exports that means you can't always swap with no code changes |
@pauldambra, got it! Thanks for answering! PostHog is great tool 👍 |
Hey @pauldambra! |
Hey, It's best to go through the in-app support flow or ask new questions rather than comment on different issues 😊 posthog-js-lite is a subset of posthog-js - you can see what it gathers here
Best to run through the support flow saying what you're observing and what you expect or open a new issue here if you think there's a bug 🙏 |
@pauldambra so currently there's no way of enabling toolbar with posthog-js-lite, correct? |
hey @joaovpmamede, no... |
@pauldambra, thanks for answering my question! Initially, I missed it |
@pauldambra got it and I really appreciate the work on the "lite" version. |
yep, that's posthog-js only 👍 |
When building the toolbar we use shared code from the main posthog app
But we never want to allow posthog-js in the toolbar, just posthog-js-lite
You can't always swap one-for-one because you can't
import { posthog } from posthog-js-lite
it'd be cool if you could
The text was updated successfully, but these errors were encountered: