Skip to content

Comments

add posthog event metric#1434

Merged
Alek99 merged 3 commits intomainfrom
carlos/add-demo-event-posthog
Jun 17, 2025
Merged

add posthog event metric#1434
Alek99 merged 3 commits intomainfrom
carlos/add-demo-event-posthog

Conversation

@carlosabadia
Copy link
Collaborator

No description provided.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Adds PostHog analytics integration to track demo form submissions, including new constants, telemetry system, and event tracking infrastructure.

  • New pcweb/telemetry/postog_metrics.py implements PostHog event tracking with potential silent failures in error handling
  • Added PostHog client v4.0.1 dependency in pyproject.toml for analytics tracking
  • Modified pcweb/pages/pricing/header.py to include demo form submission tracking
  • Inconsistent PostHog endpoint configuration between client initialization and event sending in telemetry implementation

4 files reviewed, 2 comments
Edit PR Review Bot Settings | Greptile

@Alek99 Alek99 merged commit c542e6b into main Jun 17, 2025
9 checks passed
@Alek99 Alek99 deleted the carlos/add-demo-event-posthog branch June 17, 2025 18:55
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