From e50a7d335e44aeea45165738f15886c817185762 Mon Sep 17 00:00:00 2001 From: Naisheel Verdhan Date: Fri, 23 Sep 2016 19:18:25 +0530 Subject: [PATCH] Fix README: (#126) * Fix iOS 10 specific documentation * Fix typos/spelling-errors --- README.md | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/README.md b/README.md index 9b045cd..b8799bd 100644 --- a/README.md +++ b/README.md @@ -291,7 +291,7 @@ class App extends Component { * if you just pass `notification`, you will only receive one when user resume the app. * you will not see banner if `notification->body` is not defined. - Android will receive notificaton from `FCMNotificationReceived` event - * if you pass `notification` payload. it will receive data when user click on notification + * if you pass `notification` payload, it will receive data when user click on notification * if you pass `data` payload only, it will receive data when in background e.g. fcm payload looks like: @@ -332,9 +332,9 @@ class App extends Component { ``` - When app is running in foreground - - IOS will receive notification and android **won't** (better not to do anything in foreground for hybrid and send a seprate data message.) + - IOS will receive notification and android **won't** (better not to do anything in foreground for hybrid and send a separate data message.) -NOTE: it is recommend not to rely on `data` payload for click_action as it can be overwritten (check [this](http://stackoverflow.com/questions/33738848/handle-multiple-notifications-with-gcm)). +NOTE: it is recommended not to rely on `data` payload for click_action as it can be overwritten (check [this](http://stackoverflow.com/questions/33738848/handle-multiple-notifications-with-gcm)). ## Q & A @@ -372,7 +372,7 @@ It is most likely that you are using other react-native-modules that requires co search for `compile "com.google.android.gms` in android and see who specifies specific version. Resolve conflict by loosing their version or specify a version resolve in gradle. #### How do I tell if user clicks the notification banner? -Check open from tray flag in notification. It will be either 0 or 1 for iOS and undefined or 1 for android. I decide for iOS base on [this](http://stackoverflow.com/questions/20569201/remote-notification-method-called-twice), and for android I set it if notification is triggered by intent change. +Check open from tray flag in notification. It will be either 0 or 1 for iOS and undefined or 1 for android. I decide for iOS based on [this](http://stackoverflow.com/questions/20569201/remote-notification-method-called-twice), and for android I set it if notification is triggered by intent change. #### Android notification doesn't vibrate/show head-up display etc All available features are [here](https://firebase.google.com/docs/cloud-messaging/http-server-ref#notification-payload-support). FCM may add more support in the future but there is no timeline. If you need these features now, send notification with `data` only and creating notification locally is the only way. -- 2.26.2