Skip to content
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

Trying to initialize FCM although not using Firebase #608

Closed
ori-6over6 opened this issue Jul 15, 2019 · 9 comments · Fixed by #609
Closed

Trying to initialize FCM although not using Firebase #608

ori-6over6 opened this issue Jul 15, 2019 · 9 comments · Fixed by #609

Comments

@ori-6over6
Copy link

After upgrading mixpanel from version 4+ to 5.6.4 my app crashes on launch.
The exception I'm getting is IllegalStateException: Default FirebaseApp is not initialized in this process com.sixoversix.copyglass. Make sure to call FirebaseApp.initializeApp(Context) first when calling MixpanelAPI.getInstance(context, token);
I'm not using firebase in this app so I didn't expect Mixpanel SDK to do firebase related initializations.

When debugging the code I saw that the firebase initialization is executed because the function checkPushNotificationConfiguration returns true.
When further debugging I saw it returns true because I'm using a different sdk (intercom) which internally uses firebase.

See screenshot for clarifying what I found in debug:
Screen Shot 2019-07-15 at 2 25 37 PM

Please let me know what can I do to overcome this.
Thanks!

@patedit
Copy link
Contributor

patedit commented Jul 15, 2019

Hi @ori-6over6 - wow, I was not expecting this at all. I assume that intercom includes that entry in the SDK automatically without you having to add that entry on your AndroidManifest.xml ?

@ori-6over6
Copy link
Author

Hi @patedit, yes you are correct, this service is not defined in our Manifest.

@patedit
Copy link
Contributor

patedit commented Jul 18, 2019

@ori-6over6 I just open a PR to address this issue. will be available in the next release. Thanks a lot for bringing this up!

@ori-6over6
Copy link
Author

@patedit Thanks a lot for the quick fix!
Glad I could help :)

@ori-6over6
Copy link
Author

Hey @patedit , I see a new version 5.6.5 was released but I'm still experiencing the issue...
Was the fix included the new version?

@patedit
Copy link
Contributor

patedit commented Sep 19, 2019

hi @ori-6over6 - sorry, it was not included. The PR I created won't be compatible with our react native sdk so decided to not merge it until a better fix is applied. is this affecting your production build and/or did you come up with a workaround?

@ori-6over6
Copy link
Author

Hey @patedit, we didn't find any workaround, so this issue is preventing us from upgrading our mixpanel dependency - we are still using version 4.9.4

@patedit
Copy link
Contributor

patedit commented Nov 23, 2019

@ori-6over6 please, update to v5.6.7. This should be fixed on that release. Thanks for your patience :)

@ori-6over6
Copy link
Author

@patedit updated your sdk version and the issue is resolved. Thank you :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants