Skip to content

Conversation

@maximzavadskiy
Copy link
Contributor

  • add flowplayer component usage analytics
  • bump flowplayer/player version

Copy link

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 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.

File Description
src/usage.ts New module implementing analytics tracking functionality with type definitions and event emission
src/flowplayer.tsx Integrates usage tracking by calling trackBehaviorUsage when component mounts
package.json Updates @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.

@maximzavadskiy maximzavadskiy force-pushed the fix/PLAY-1164/usage_analytics branch from 37a20f0 to 6ff2cd2 Compare September 18, 2025 10:12
@maximzavadskiy maximzavadskiy changed the title fix(PLAY-1164): add flowplayer component usage analytics fix(PLAY-1164): track Flowplayer component feature adoption event Sep 18, 2025
@maximzavadskiy maximzavadskiy marked this pull request as ready for review September 18, 2025 10:14
@maximzavadskiy maximzavadskiy force-pushed the fix/PLAY-1164/usage_analytics branch from 6ff2cd2 to bac6e48 Compare September 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
Copy link
Contributor

@ondreian ondreian left a comment

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 main Sep 19, 2025
1 check passed
@maximzavadskiy maximzavadskiy deleted the fix/PLAY-1164/usage_analytics branch September 19, 2025 13:06
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.

3 participants