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
Feature/modules #24
Feature/modules #24
Conversation
Could this be a problem because of the 2.8.+ dependency?
I didn't change anything specifically but gradle has been a huge PITA :(.
…On Wed, 12 Sep 2018 at 15:46, Cory Zachman ***@***.***> wrote:
opening it up i get these build errors? anything specific to your
environment that you changed?
[image: image]
<https://user-images.githubusercontent.com/447500/45429135-f7042000-b6a2-11e8-885d-1d33e8c69dcd.png>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#24 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAFeUDHyGSaqMGid2BgSSPWqMFQZ9G1Jks5uaRAlgaJpZM4WlWMt>
.
|
@@ -75,7 +75,7 @@ public EventData(BitmovinAnalyticsConfig bitmovinAnalyticsConfig, String impress | |||
this.path = bitmovinAnalyticsConfig.getPath(); | |||
this.experimentName = bitmovinAnalyticsConfig.getExperimentName(); | |||
this.playerTech = Util.getPlayerTech(); | |||
this.setUserAgent(Util.getUserAgent(bitmovinAnalyticsConfig.getContext())); | |||
this.setUserAgent("unknown"); //Util.getUserAgent(bitmovinAnalyticsConfig.getContext())); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you plan to find a solution here or will it stay "unknown"
for now?
@@ -0,0 +1,5 @@ | |||
package com.bitmovin.analytics.adapters; | |||
|
|||
public class PlayerAdapterFactory { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leftover?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good to me! Would be good to test it in a real world setup:
- Android app with dependencies: ExoPlayer, analytics core, analytics Exo
- Android app with dependencies: BitmovinPlayer, analytics core, analytics BitmovinPlayer
@hawk23 the two test apps in the repo do exactly that .. |
…h-filters-to-be-set feature/updte-contenful-link-with-filters-to-be-set
Changed the modules around to have a bitmovin-analytics-exoplayer and bitmovin-analyltics-bitmovin-player package