Commit 09f4f2e0 authored by Libin Lu's avatar Libin Lu Committed by GitHub

Update README.md

parent 16b595b0
...@@ -330,16 +330,15 @@ Since Lollipop, the push notification icon is required to be all white, otherwis ...@@ -330,16 +330,15 @@ Since Lollipop, the push notification icon is required to be all white, otherwis
#### I am using Proguard #### I am using Proguard
You need to add this to your `android/app/proguard-rules.pro`: You need to add this to your `android/app/proguard-rules.pro`:
#### 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.
``` ```
# Google Play Services # Google Play Services
-keep class com.google.android.gms.** { *; } -keep class com.google.android.gms.** { *; }
-dontwarn com.google.android.gms.** -dontwarn com.google.android.gms.**
``` ```
#### 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.
#### Android notification doesn't vibrate/show head-up display etc #### 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 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