aws-amplify / amplify-js

A declarative JavaScript library for application development using cloud services.
https://docs.amplify.aws/lib/q/platform/js
Apache License 2.0
9.43k stars 2.13k forks source link

Add badge count to push notifications #11596

Open kevinxu3 opened 1 year ago

kevinxu3 commented 1 year ago

Is this related to a new or existing framework?

React Native

Is this related to a new or existing API?

Push Notifications

Is this related to another service?

No response

Describe the feature you'd like to request

Hi, I found an issue with notifications not working with badge count. I am sending the badge count to the app from the backend but I am not receiving that count. I understand in the foreground notifications are not shown, but we should still allow silent notifications that update badge count. If this is not possible, we should show the badge count so we can update it using the amplify update badge count function.

This was working previously on the old Push Notifications by the way.

Describe the solution you'd like

Allow push notifications to send silent notifications in the foreground and pass badge count information in the notification objects for listener functions.

Describe alternatives you've considered

None. This was working previously before and currently there is no alternative. Only solution right now is to not use badge count so the badge count is no inaccurate.

Additional context

Somewhat related to this issue (https://github.com/aws-amplify/amplify-js/issues/11564)

Is this something that you'd be interested in working on?

cwomack commented 1 year ago

@kevinxu3, thank you for creating this separate issue! Similar to the ThreadId issue you opened, I'll mark this as a feature request and review this with the team regarding a possible regression from previous versions of Push Notifications with the badge count.

kevinxu3 commented 11 months ago

Is there any update with this?

kevinxu3 commented 11 months ago

@cwomack Hi is there any update on this? This is still a blocker for us.