bitmovin / bitmovin-player-android-analytics-conviva

Integration of the Bitmovin Android Player SDK with the Conviva Analytics
MIT License
1 stars 4 forks source link

Allow for registering a new ConvivaAnalyticsIntegration to a reused Player #52

Closed kvolkers closed 1 year ago

kvolkers commented 1 year ago

The current setup expects for 1 ConvivaAnalyticsIntegration instance to be used with 1 BitmovinPlayer instance.

To make it easier to get Conviva to track correctly in our implementation when changing video stream with 1 BitmovinPlayer instance (live channel zapping, play next, some use cases with chromecast) we'd like to use a new ConvivaAnalyticsIntegration for each new video stream.

The following changes were made:

I couldn't get all tests to work, but I suspect this might be an issue with the 2 tests itself, report included. image androidTests.zip

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

dweinber commented 1 year ago

For the record: I merged this into develop and re-ran the tests on my main dev device successfully: image