Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notifications.events().registerNotificationOpened() and RNN 4.2.3 #833

Closed
claudiozam opened this issue Mar 2, 2022 · 4 comments
Closed
Assignees

Comments

@claudiozam
Copy link

Hi, after the new release 4.2.3 (#828) the behavior of the implementation changed.

This is the new issue.

Open de App
Go to background (Do not kill de app)
Send new Push notification
Tap on the notification (It will open de app).
Go to background (Do not kill de app)
Send new Push notification and do not tap/open the new notification
Open the app.
It will execute Notifications.events().registerNotificationOpened() with the last notification payload.

With RNN 4.2.2 i don't have this behavior

Enviroment

"react-native": "0.67.3"
"react-native-notifications": "4.2.3"

Thanks

@DanielEliraz
Copy link
Collaborator

which android version?

@claudiozam
Copy link
Author

Android 12 @DanielEliraz

@DanielEliraz
Copy link
Collaborator

DanielEliraz commented Mar 9, 2022

@claudiozam, if you are using targetSdk < 31 it should be fixed by #837. please check it with 4.2.4

@DanielEliraz DanielEliraz self-assigned this Mar 9, 2022
@claudiozam
Copy link
Author

claudiozam commented Mar 9, 2022

Yes, targetSdk = 30

Samsung S8+ (Anndroid 9) - OK
Samsung S10+ (Android 12) - OK
Emulator (Android 11) - OK

Works excellent

Thanks. @DanielEliraz !!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants