fix promise error

b3d97865 · Libin Lu · 5658029f · b3d97865
Commit b3d97865 authored Sep 12, 2018 by Libin Lu
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

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

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
@@ -211,8 +211,8 @@ public class FIRMessagingModule extends ReactContextBaseJavaModule implements Li
    }

    @ReactMethod
-    public void subscribeToTopic(String topic){
-	try {
+    public void subscribeToTopic(String topic, Promise promise){
+	    try {
            FirebaseMessaging.getInstance().subscribeToTopic(topic);
            promise.resolve(null);
        } catch (IOException e) {
@@ -222,8 +222,8 @@ public class FIRMessagingModule extends ReactContextBaseJavaModule implements Li
    }

    @ReactMethod
-    public void unsubscribeFromTopic(String topic){
-	try {
+    public void unsubscribeFromTopic(String topic, Promise promise){
+	    try {
            FirebaseMessaging.getInstance().unsubscribeFromTopic(topic);
            promise.resolve(null);
        } catch (IOException e) {