wix / react-native-notifications

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

fix: add pending intent immutable flag to NotificationIntentAdapter #921

Closed doug-sheridan closed 7 months ago

doug-sheridan commented 1 year ago

Google Play's target API level requirement increased to API 31: https://developer.android.com/google/play/requirements/target-sdk#pre12

One of the requirements is to always specify the mutability of each PendingIntent object: https://developer.android.com/google/play/requirements/target-sdk#pre12:~:text=Pending%20intent%20mutability%3A%20You%20must%20specify%20the%20mutability%20of%20each%20PendingIntent%20object%20that%20your%20app%20creates.

This PR adds the PendingIntent.FLAG_IMMUTABLE to the only place where it was needed.

ViniciusMargotti commented 1 year ago

I'm having the same issues with upgrading to api 31, awaiting fix PendingIntent.FLAG_IMMUTABLE

ViniciusMargotti commented 1 year ago

We identified the problem was in the "react-native-notifications" library

https://github.com/invertase/react-native-firebase/issues/6759