-
Notifications
You must be signed in to change notification settings - Fork 13
Open
Description
I was looking through the implementation of this integration and noticed that it does not override Integration.reset().
Would it make sense to reset the Firebase client in this method? I was thinking either by calling FirebaseAnalytics.resetAnalyticsData() or simply setting the user ID to null with FirebaseAnalytics.setUserId(null). This way, the user can be cleared when they log out by calling Segment's Analytics.with(context).reset()
@Override
public void reset() {
super.reset();
firebaseAnalytics.resetAnalyticsData();
OR
firebaseAnalytics.setUserId(null);
logger.verbose("firebaseAnalytics.reset();");
}
See:
Metadata
Metadata
Assignees
Labels
No labels