-
-
Notifications
You must be signed in to change notification settings - Fork 13
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(platform): Platform Telemetry #2459
Conversation
apps/console/.dev.vars.example
Outdated
@@ -5,3 +5,6 @@ INTERNAL_GOOGLE_ANALYTICS_TAG = "G-NHNH4KRWC3" | |||
STRIPE_API_SECRET = "" | |||
STRIPE_WEBHOOK_SECRET = "" | |||
STRIPE_PRO_PLAN_ID = "" | |||
|
|||
REACT_APP_PUBLIC_POSTHOG_KEY="" |
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.
POSTHOG_PUBLIC_KEY
apps/console/.dev.vars.example
Outdated
@@ -5,3 +5,6 @@ INTERNAL_GOOGLE_ANALYTICS_TAG = "G-NHNH4KRWC3" | |||
STRIPE_API_SECRET = "" | |||
STRIPE_WEBHOOK_SECRET = "" | |||
STRIPE_PRO_PLAN_ID = "" | |||
|
|||
REACT_APP_PUBLIC_POSTHOG_KEY="" | |||
REACT_APP_PUBLIC_POSTHOG_HOST="" |
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.
POSTHOG_HOST
dab784a
to
1f071f2
Compare
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.
Define new environment variables in GitHub workflow configuration files.
Is it possible to match platform/posthog/tsconfig.json
with others?
}, | ||
} | ||
|
||
await fetch(`${POSTHOG_PROXY_HOST}/e/`, init) |
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.
It is a good idea to just return here instead. Let the caller decide.
properties, | ||
}: { | ||
eventName: string | ||
apiKey: string |
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.
Is this the only parameter representing the environment?
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.
Yeah, only one API key per app in PostHog
Our proxy worker sends requests with specific API key and then it gets sorted in posthog
Description
Sets up PostHog for Analytics
Related Issues
Testing
TBD
Checklist