search

urbanairship / android-library

Urban Airship Android SDK
Other
109 stars 123 forks source link

Remove check for user notifications enabled when checking opt in status #190

Closed ds8k closed 2 years ago

ds8k commented 3 years ago

What do these changes do?

Removes the check for user notifications enabled when checking opt in status

Why are these changes necessary?

Discrepancy between Android and iOS library. iOS returns system status without this check. Removing it gives us consistent behavior of isUserNotificationsEnabled and IsUserNotificationsOptedIn across both platforms.

Anything else a reviewer should know?

Resolves https://github.com/urbanairship/react-native-module/issues/349 and https://github.com/urbanairship/android-library/issues/189

rlepinski commented 2 years ago

Keeping as is, resolved issue in the react-native plugin