Skip to content
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

Add comprehensive analytics tracking #5271

Merged
merged 15 commits into from
Dec 2, 2024
Merged

Add comprehensive analytics tracking #5271

merged 15 commits into from
Dec 2, 2024

Conversation

xingyaoww
Copy link
Collaborator

@xingyaoww xingyaoww commented Nov 25, 2024

This PR adds comprehensive analytics tracking for user entry points and first message:

  • Added tracking for GitHub repository selection
  • Added tracking for ZIP file upload
  • Added tracking for first message after context (with entry point info)
  • Added analytics documentation in docs/analytics.md

The changes will help us better understand how users enter and use the application.


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:dbb3902-nikolaik   --name openhands-app-dbb3902   docker.all-hands.dev/all-hands-ai/openhands:dbb3902

@amanape
Copy link
Member

amanape commented Dec 2, 2024

Something seems wrong with the conflict resolver. The chat interface has been moved (https://github.com/All-Hands-AI/OpenHands/blob/main/frontend/src/components/features/chat/chat-interface.tsx) and this PR is trying to create a new one in its old place.

@xingyaoww xingyaoww force-pushed the add-analytics-tracking branch from 2c0ebb5 to b6aa786 Compare December 2, 2024 16:55
@xingyaoww
Copy link
Collaborator Author

I tried to manually resolve merge conflict and it seems OpenHands did a way better job than me do 😓

@xingyaoww xingyaoww merged commit 2f11634 into main Dec 2, 2024
14 checks passed
@xingyaoww xingyaoww deleted the add-analytics-tracking branch December 2, 2024 18:02
@enyst
Copy link
Collaborator

enyst commented Dec 5, 2024

The initial post says:

Added analytics documentation in docs/analytics.md

The PR didn't add this, though. Was it forgotten or postponed? @xingyaoww

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.

5 participants