(push-notifications/android) Open foreground push notification with context

[abennouna]

Currently in the Push Notifications plugin 4.1.2, the foreground notifications, i.e. the ones that are delivered while the app in active and in foreground, are shown in Android, but:

• do not disappear on tap
• do not trigger the pushNotificationActionPerformed listener
• the icon color cannot be customized

I succeeded in performing that 3 features that are needed for me anyway.

The result are here:

• to have tap makes foreground notif disappear --> abennouna/capacitor-plugins@dabc089
• to customize icon color --> abennouna/capacitor-plugins@c83206a (also see abennouna/capacitor-plugins@a63d66d to have the same color for both background and foreground notifs)
• trigger the pushNotificationActionPerformed listener --> abennouna/capacitor-plugins@d9ec714

I'm talking only about Android here. iOS works just fine as documented.

Do you think everyone would need the 3 features, or one of them? Should I open a PR for any?

[abennouna]

Update: abennouna/capacitor-plugins@5f49e40

Two PRs already address the tap issue and pushNotificationActionPerformed issue:

• feat(push-notifications): allow more control over foreground notifications capacitor-plugins#1423
• fix(push-notification): notification not dispatching click when received in foreground capacitor-plugins#1478

There is also this older PR:

• fix(pushnotifications): App Foreground -> Notification not-clickable capacitor-plugins#1324

Well basically the solution has been around from some time.

[RaddishIoW]

I've published my own plugin after realising that the PRs are just being ignored.

It's a drop-in replacement for the official plugin. It's released as a beta at the mo, just because I've only quickly tested it with my project...
@haylltd/capacitor-push-notifications

I'd welcome any community PRs for extra features or bug fixes.

[facusatoshi]

Hi I will to test your version cause i need the features works on android.
Do you think that your pull requests will be approved?

Regards

[abennouna]

I settled on using patch-package instead of waiting for PRs to be merged, and instead of using another plugin.