Could you add a "Tapped" event?

Question

Could you add a "Tapped" event?

Closed this issue 9 years ago · 3 comments

First of all, very nice library however i'm still missing a tap event when the user taps the notification.

Answer 1 · 2016-04-14T13:45:50.000Z

Hey Steven,

The LNRNotificationManager method showNotification:title:body:callback: has an admittedly poorly-named "callback" block parameter that is called when a user taps on your notification. I'll change that to "onTap" in the next few days when I have some time to make things clearer but it is implemented. If it doesn't quite meet your needs do you have any recommendations on what might make it more useful for you?

Answer 2 · 2016-04-14T14:00:32.000Z

Hey, thanks for the fast response, implemented & working :), used following code :
```
notificationManager.showNotification(title, body: body, onTap: { () -> Void in
print("Notification tapped")
})

Answer 3 · 2016-04-14T14:31:56.000Z

Just so you know, I updated the callback parameter name to "onTap" in version 0.4.1, which I just pushed to the spec repo a few minutes ago.