Mixpanel Screen API about analytics-android

[f2prateek]

We call in track just a line above https://github.com/segmentio/analytics-android/blob/master/analytics/src/main/java/com/segment/android/integrations/MixpanelIntegration.java#L129.

The code you linked tracks a charge only the revenue is defined.

[naqi]

That would be correct if I was calling an event.I am calling screen which
is calling event for mixpanel.
On Jul 10, 2014 4:17 PM, "Prateek Srivastava" [email protected]
wrote:

We call in track just a line above
https://github.com/segmentio/analytics-android/blob/master/analytics/src/main/java/com/segment/android/integrations/MixpanelIntegration.java#L129
.

The code you linked tracks a charge only the revenue is defined.

—
Reply to this email directly or view it on GitHub
#58 (comment)
.

[f2prateek]

What should it be calling instead? Mixpanel only supports track as far as I can see. We simply proxy the screen calls to their track since that's what they support.

I'm also not sure where your'e thinking that we send a screen event only for revenue in the code you posted.

[f2prateek]

@naqi Do you still think this is the wrong behavior? What do you expect the right one to be?

[naqi]

@f2prateek I ended up making the changes myself. Will send a pull request some time after this week.

[f2prateek]

@naqi Sweet!

[f2prateek]

@naqi Mind sharing the changes? We wanted to verify what the behaviour should be, and correct our iOS project too since this is what we do there as well.