-
Notifications
You must be signed in to change notification settings - Fork 363
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
prefix device-specific distinct IDs with '$device:' #810
Conversation
@@ -14,6 +14,7 @@ | |||
import com.mixpanel.android.util.HttpService; | |||
import com.mixpanel.android.util.RemoteService; | |||
|
|||
import org.hamcrest.CoreMatchers; |
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.
nit: redundant
@@ -1779,7 +1777,7 @@ public void identify(String distinctId) { | |||
MPLog.e(LOGTAG, "Can't identify with null distinct_id."); | |||
return; | |||
} | |||
if (distinctId != mPersistentIdentity.getEventsDistinctId()) { | |||
if (!distinctId.equals(mPersistentIdentity.getEventsDistinctId())) { |
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.
good catch!
README.md
Outdated
@@ -158,7 +158,7 @@ We'd also love for you to come and work with us! Check out our **[opening positi | |||
<a name="changelog"></a> | |||
# Changelog | |||
|
|||
See [wiki page](https://github.com/mixpanel/mixpanel-android/wiki/Changelog). | |||
See [wiki page](CHANGELOG.md). |
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.
See [wiki page](CHANGELOG.md). | |
See [changelog](CHANGELOG.md). |
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.
LGTM!
@carstenhag mixpanel-swift started prefixing The Objective-C SDK started w/ v5.0.2: https://github.com/mixpanel/mixpanel-iphone/releases/tag/v5.0.2 Both were released 5 months ago. If you are using a version >= those, then I would recommend reaching out to support@mixpanel.com for help troubleshooting any identity management issues you're experiencing. |
No description provided.