Merge pull request #96 from jasongaare/master

yonahforst · web-flow · commit 2ab7e5fa1497 · 2017-04-28T10:15:54.000-04:00
Add notification support for Android (getPermissionStatus only)
diff --git a/ReactNativePermissions.js b/ReactNativePermissions.js
@@ -78,6 +78,9 @@ class ReactNativePermissions {
 		} else if (permission == 'location') {
 			options = type || 'whenInUse'
 		} else if (permission == 'notification') {
+			if (Platform.OS === 'android') {
+				return Promise.reject(`ReactNativePermissions: notification cannot be requested on Android`)
+			}
 			options = type || ['alert', 'badge', 'sound']
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -78,6 +78,9 @@ class ReactNativePermissions {`
`78`	`78`	`} else if (permission == 'location') {`
`79`	`79`	`options = type \|\| 'whenInUse'`
`80`	`80`	`} else if (permission == 'notification') {`
	`81`	`+ if (Platform.OS === 'android') {`
	`82`	+ return Promise.reject(`ReactNativePermissions: notification cannot be requested on Android`)
	`83`	`+ }`
`81`	`84`	`options = type \|\| ['alert', 'badge', 'sound']`
`82`	`85`	`}`
`83`	`86`