Skip to content

Conversation

@urmauur
Copy link
Member

@urmauur urmauur commented Oct 11, 2025

Describe Your Changes

This pull request simplifies the ChatInput component by removing all product analytics tracking logic related to PostHog. The changes focus on cleaning up unnecessary imports and code for tracking message send events.

Code cleanup:

  • Removed imports of useAnalytic and posthog-js from ChatInput.tsx, as analytics tracking is no longer needed.
  • Removed the productAnalytic hook and all related logic for tracking message send events with PostHog, including the conditional event capture and error handling. [1] [2]

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@urmauur urmauur added this to the v0.7.2 milestone Oct 11, 2025
@urmauur urmauur self-assigned this Oct 11, 2025
Copilot AI review requested due to automatic review settings October 11, 2025 14:47
@urmauur urmauur added this to Jan Oct 11, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes product analytics tracking functionality from the ChatInput component by eliminating all PostHog-related code and imports. The changes streamline the component by removing unnecessary analytics dependencies and tracking logic.

  • Removed PostHog imports and analytics hook usage
  • Eliminated message send event tracking logic
  • Cleaned up related error handling for analytics

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@urmauur urmauur requested a review from Minh141120 October 13, 2025 03:11
Copy link
Member

@Minh141120 Minh141120 left a comment

Choose a reason for hiding this comment

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

LGTM!

@urmauur urmauur merged commit 7b5060c into dev Oct 13, 2025
21 checks passed
@urmauur urmauur deleted the chore/disable-posthog-event branch October 13, 2025 03:13
@github-project-automation github-project-automation bot moved this to QA in Jan Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: QA

Development

Successfully merging this pull request may close these issues.

3 participants