Azure / azure-notificationhubs-android

Android SDK and Samples for Azure Notification Hubs
Apache License 2.0
33 stars 63 forks source link

Removing explict FirebaseReceiver start #119

Closed marstr closed 4 years ago

marstr commented 4 years ago

Description

Remove the NotificationHub global single instance's explicit call to start the FirebaseReceiver. This should spare our customers a exception that gets thrown if it is called from the background, while also removing a small piece of coupling between the two classes.

Related PRs or issues

Fixes #116

Misc

I tested in the emulator using the Java sample application, and confirmed that we still receive notifications in both the foreground and background.