Merge pull request #1040 from punksta/patch-1

Safe FirebaseMessaging usage

Merge pull request #1040 from punksta/patch-1
Safe FirebaseMessaging usage
5cec691d · Libin Lu · GitHub · 7557b00e · 40c3f7ee · 5cec691d
Commit 5cec691d authored Sep 07, 2018 by Libin Lu Committed by GitHub Sep 07, 2018
Show whitespace changes
Inline Side-by-side

Showing with 14 additions and 2 deletions

android/src/main/java/com/evollu/react/fcm/FIRMessagingModule.java ...rc/main/java/com/evollu/react/fcm/FIRMessagingModule.java +14 -2

No files found.
--- 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 class FIRMessagingModule extends ReactContextBaseJavaModule implements Li

    @ReactMethod
    public void subscribeToTopic(String 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){
+	try {
            FirebaseMessaging.getInstance().unsubscribeFromTopic(topic);
+            promise.resolve(null);
+        } catch (IOException e) {
+            e.printStackTrace();
+            promise.reject(null,e.getMessage());
+        }
    }

    @ReactMethod