Skip to content

fix(PLAY-1164): track Flowplayer component feature adoption event - #63

Merged
maximzavadskiy merged 2 commits into
mainfrom
fix/PLAY-1164/usage_analytics
Sep 19, 2025
Merged

fix(PLAY-1164): track Flowplayer component feature adoption event#63
maximzavadskiy merged 2 commits into
mainfrom
fix/PLAY-1164/usage_analytics

Conversation

@maximzavadskiy

Copy link
Copy Markdown
Contributor
  • add flowplayer component usage analytics
  • bump flowplayer/player version

CopilotAI left a comment

Copy link
Copy Markdown

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 adds analytics tracking functionality to the react-flowplayer component and updates the underlying flowplayer dependency. The main purpose is to track component usage through a new analytics system that emits events when the flowplayer component is mounted.

  • Introduces a usage tracking system with configurable sample rates
  • Integrates tracking into the Flowplayer component to emit "flowplayer-component-mounted" events
  • Updates @flowplayer/player dependency from version 3.21.0 to 3.31.0

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

FileDescription
src/usage.tsNew module implementing analytics tracking functionality with type definitions and event emission
src/flowplayer.tsxIntegrates usage tracking by calling trackBehaviorUsage when component mounts
package.jsonUpdates @flowplayer/player dependency to version 3.31.0

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

Comment threadsrc/flowplayer.tsx Outdated
@maximzavadskiy
maximzavadskiyforce-pushed the fix/PLAY-1164/usage_analytics branch from 37a20f0 to 6ff2cd2CompareSeptember 18, 2025 10:12
@maximzavadskiymaximzavadskiy changed the title fix(PLAY-1164): add flowplayer component usage analyticsfix(PLAY-1164): track Flowplayer component feature adoption eventSep 18, 2025
@maximzavadskiy
maximzavadskiy marked this pull request as ready for review September 18, 2025 10:14
@maximzavadskiy
maximzavadskiyforce-pushed the fix/PLAY-1164/usage_analytics branch from 6ff2cd2 to bac6e48CompareSeptember 18, 2025 11:49
@maximzavadskiy
maximzavadskiy marked this pull request as draft September 19, 2025 12:32
@maximzavadskiy
maximzavadskiy marked this pull request as ready for review September 19, 2025 13:03

@ondreianondreian left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:shipit:

@maximzavadskiy
maximzavadskiy merged commit 1d462a8 into mainSep 19, 2025
1 check passed
@maximzavadskiy
maximzavadskiy deleted the fix/PLAY-1164/usage_analytics branch September 19, 2025 13:06
Sign up for freeto 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.

3 participants

@maximzavadskiy@ondreian