benwilkins / laravel-fcm-notification

Laravel FCM (Firebase Cloud Messaging) Notification Channel
MIT License
213 stars 91 forks source link

Use the same default priority as Firebase #5

Closed nicoqh closed 6 years ago

nicoqh commented 7 years ago

Firebase notification messages default to high priority.

Should this library do the same?

thomaswardiii commented 6 years ago

Where does it say this? The priority seems to be dependent upon the type of message and its purpose, not that all of them are high priority for all types of notifications.

nicoqh commented 6 years ago

At the time of writing I think this library always included the notification key, in which case the message is considered a "notification message" by Firebase and has a high priority by default.

But this is mute now, so I'm closing.