Skip to content

feat(shared): track user hover card analytics#5596

Merged
rebelchris merged 3 commits intomainfrom
eng-829-add-analytics-event-for-user-hover-cards
Feb 26, 2026
Merged

feat(shared): track user hover card analytics#5596
rebelchris merged 3 commits intomainfrom
eng-829-add-analytics-event-for-user-hover-cards

Conversation

@rebelchris
Copy link
Contributor

@rebelchris rebelchris commented Feb 25, 2026

Summary

  • add LogEvent.HoverUserCard = 'hover user card' in shared log events
  • emit hover-card analytics from ProfileTooltip when tooltip becomes visible via onShow
  • include target_id: userId so analytics can attribute hover-card impressions to the hovered user
  • add focused tests for ProfileTooltip analytics logging and custom tooltip.onShow passthrough

Key Decisions

  • instrumented ProfileTooltip as the single integration point to cover all user hover-card surfaces
  • fired on tooltip onShow (not raw mouse enter) to avoid noisy accidental hover events
  • kept implementation minimal and aligned with existing hover analytics patterns

Closes ENG-829


Created by Huginn 🐦‍⬛

Preview domain

https://eng-829-add-analytics-event-for.preview.app.daily.dev

@vercel
Copy link

vercel bot commented Feb 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Feb 26, 2026 7:34am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Feb 26, 2026 7:34am

Request Review

@rebelchris rebelchris merged commit 89a6b01 into main Feb 26, 2026
3 of 5 checks passed
@rebelchris rebelchris deleted the eng-829-add-analytics-event-for-user-hover-cards branch February 26, 2026 07:30
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.

1 participant