search

wix / react-native-notifications

React Native Notifications
MIT License
3.25k stars 764 forks source link

Change the way the initial notification is extracted from intent at #966

Closed kristofb closed 1 year ago

kristofb commented 1 year ago

application startup.

Tested on different Android versions up to 13

skinsapp commented 1 year ago

Well done! This works for me on Android 13. Without it, Notifications.getInitialNotification absolutely does not work.

Please merge.

greenimpala commented 1 year ago

For me this works on the very first notification when the app in installed. Once the app is open and force quit, subsequent notifications end up undefined again.

kristofb commented 1 year ago

I just rebased my branch on master. No more changes, for me, this hotfix is working well based on master (release 4.3.5)

DanielEliraz commented 1 year ago

@kristofb what about the case @greenimpala wrote:

For me this works on the very first notification when the app in installed. Once the app is open and force quit, subsequent notifications end up undefined again.

DanielEliraz commented 1 year ago

hey, this change was added in #983 and released in 5.0.0. please verify it