[Bug]: NotificationClickEvent returning `launchUrl` instead of `launchURL` (on Android) about react-native-onesignal HOT 7 CLOSED

I created an issue here some months ago: #1635
It was marked as resolved 8h ago so I hope new update will rollout soon!

from react-native-onesignal.

@Aya-Sato @pilotpirxie released
https://github.com/OneSignal/react-native-onesignal/releases/tag/5.2.1

from react-native-onesignal.

@Martinocom-Switcho Thank you for letting us know! I will close the issue

from react-native-onesignal.

@pilotpirxie @Martinocom-Switcho
Yes, I closed the issue because it was a duplicate. I do agree with @Martinocom-Switcho though that the original shouldn't have been closed if it was not released yet..

from react-native-onesignal.

I have the same issue and I had to do the ugly hack to quiet warnings and the error to support both platforms. On iOS it works fine, the object returned to the android device is invalid.

      if (
        oneSignalEvent.notification?.launchURL ||
        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
        // @ts-expect-error
        oneSignalEvent.notification?.launchUrl
      ) {}

from react-native-onesignal.

@Aya-Sato Shouldn't you first release the fix and then close? When can we expect change to be released?

from react-native-onesignal.

@pilotpirxie I think it was closed because #1635 was still open and this was a duplicate.

But I agree that if merged ≠ released, the original #1635 should't be closed. Or at least it should be stated somewhere if this is the policy of this repo.

from react-native-onesignal.