phonegap / phonegap-plugin-push

Register and receive push notifications
MIT License
1.94k stars 1.91k forks source link

Clearing the Inbox Stack on Android #1280

Open Caine72 opened 8 years ago

Caine72 commented 8 years ago

Expected Behaviour

Is there any way of clearing the Inbox Stack for Android? I was expecting the push.clearAllNotifications() to do this. Is there an alternate way?

My app sends out multiple notifications today and clears notifications and badge up on launch. If i have opened the app im expecting all notification indications to reset to 0.

Actual Behaviour

Same behaviour as mentioned as intended in:[(https://github.com/phonegap/phonegap-plugin-push/issues/924)]

Reproduce Scenario (including but not limited to)

Send notification 1. Open application. Badge is reset ( set badge to zero) Notifications are cleared (clear all clearAllNotifications()) Close app. Badge is now empty and notifications are no longer visible, Send notification 2. Badge presents numer 1. Inbox stack contains both notifiction 1 and 2.

Steps to Reproduce

Platform and Version (eg. Android 5.0 or iOS 9.2.1)

Android 6.0.1

(Android) What device vendor (e.g. Samsung, HTC, Sony...)

Sony + Nova Launcher.

Cordova CLI version and cordova platform version

cordova --version                                    # 6.3.1
cordova platform version android                     #  5.2.2

Plugin version

1.8.0

Sample Push Data Payload

Sample Code that illustrates the problem

Logs taken while reproducing problem

macdonst commented 8 years ago

@Caine72 not enough information. You haven't show me your init code. Are you setting android.clearBadge and/or android.clearNotifications? What does your push payload look like? Are you sending a distinct notId with each push? Are they inbox or regular style?

Caine72 commented 8 years ago

Hi! @macdonst Been away on a business trip. Will add more info before end of day. But until then. android.clearBadge : yes. android.clearNotifications yes. notId: I have been experimenting, tested both with and without. Set it as a random millisec string. style: inbox. More or less same setup as issue 924 i was linking too in original post.

Caine72 commented 8 years ago

As i said earlier i have experimented with both clearing the notifications manually and by init-code. At the moment the payload does not include the notId. Adding notId and/or testing with init-states/manually clearing made no difference. The inbox queue size keeps on building as one does not click the notification/s specifically.

So I would just like a clarification if clearing the Inbox style is supposed to be clear:able with the clearAllNotifcations function... or at all for that matter. As it right now acts more like a discard, than a clear.

Init code.

 "android": {
    "icon": "ic_stat",
     "iconColor": "#4e7987",
     "clearNotifications": "true",
     "clearBadge":"true"
 }

Payload:

 android": {
             "data": {
                "title": "myTitle",
                "message": "myMessage",
                "style": "inbox",
                "content-available" : "1"
             }
         }
Caine72 commented 8 years ago

@macdonst Any feedback regarding this? =) Anything you want me to add or expand upon?

macdonst commented 8 years ago

@Caine72 nope, just haven't had a chance to get to it. Traveling myself.

stale[bot] commented 6 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.