Merge pull request #1040 from punksta/patch-1

Safe FirebaseMessaging usage
evollu · Sep 7, 2018 · 5cec691 · 5cec691
2 parents 7557b00 + 40c3f7e
commit 5cec691
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/android/src/main/java/com/evollu/react/fcm/FIRMessagingModule.java b/android/src/main/java/com/evollu/react/fcm/FIRMessagingModule.java
@@ -212,12 +212,24 @@ public void removeAllDeliveredNotifications(){
 
     @ReactMethod
     public void subscribeToTopic(String topic){
-        FirebaseMessaging.getInstance().subscribeToTopic(topic);
+	try {
+            FirebaseMessaging.getInstance().subscribeToTopic(topic);
+            promise.resolve(null);
+        } catch (IOException e) {
+            e.printStackTrace();
+            promise.reject(null,e.getMessage());
+        }
     }
 
     @ReactMethod
     public void unsubscribeFromTopic(String topic){
-        FirebaseMessaging.getInstance().unsubscribeFromTopic(topic);
+	try {
+            FirebaseMessaging.getInstance().unsubscribeFromTopic(topic);
+            promise.resolve(null);
+        } catch (IOException e) {
+            e.printStackTrace();
+            promise.reject(null,e.getMessage());
+        }
     }
 
     @ReactMethod