Merge pull request #34 from mvila/master

Make Android requestPermission() method compatible with RN 0.33

android/src/main/java/com/joshblour/reactnativepermissions/ReactNativePermissionsModule.java

@@ -10,6 +10,7 @@ import android.support.v4.content.PermissionChecker;
 
 import com.facebook.react.bridge.Callback;
 import com.facebook.react.bridge.Promise;
+import com.facebook.react.bridge.PromiseImpl;
 import com.facebook.react.bridge.ReactApplicationContext;
 import com.facebook.react.bridge.ReactContextBaseJavaModule;
 import com.facebook.react.bridge.ReactMethod;
@@ -79,13 +80,19 @@ public class ReactNativePermissionsModule extends ReactContextBaseJavaModule {
   @ReactMethod
   public void requestPermission(final String permissionString, String nullForiOSCompat, final Promise promise) {
     String permission = permissionForString(permissionString);
-    mPermissionsModule.requestPermission(permission, new Callback() {
+    Callback resolve = new Callback() {
       @Override
       public void invoke(Object... args) {
         getPermissionStatus(permissionString, promise);
-//        promise.resolve((boolean)args[1] ? "authorized" : "denied");
       }
-    }, null);
+    };
+    Callback reject = new Callback() {
+      @Override
+      public void invoke(Object... args) {
+        // NOOP
+      }
+    };
+    mPermissionsModule.requestPermission(permission, new PromiseImpl(resolve, reject));
   }