onPushNotification(title,description) needs payload

[chrisjenx]

Hey love the lib,

But ideally public fun onPushNotification(title: String?, body: String?) {} should also take nullable payload.

Most cases one would filter these messages and decide to show them based on if the user is logged in (pushes include userId).

Theres a couple of quick fixes:

public fun onPushNotification(title: String?, body: String?, data: PayloadData?) {}

Or you can pipe the notification title/body into the payload (which is what I used to do)

val payloadData = message.data.toMutableMap()
message.notification?.also {
  payloadData["title"] = it.title ?: payloadData["title"]
  payloadData["body"] = it.title ?: payloadData["body"]
}

notifierManager.onPushPayloadData(data)

Or you can pass the notification to onPushPayloadData(title, body, data)

The TLDR is that these shouldn't be seperate methods, it should be up to the consumer how the handle show incoming messages.

[mirzemehdi]

Implemented in #110