Commit e50a7d33 authored by Naisheel Verdhan's avatar Naisheel Verdhan Committed by Libin Lu

Fix README: (#126)

* Fix iOS 10 specific documentation
* Fix typos/spelling-errors
parent aaf2017a
......@@ -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.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment