PushNotificationIOS.removeEventListener method signature requires unused handler Function

[schumannd]

react-native/Libraries/PushNotificationIOS/PushNotificationIOS.js

Line 236 in 726d001

handler: Function,

This throws unhelpful flow errors. If this is an error, I can provide a PR removing this line.

[react-native-bot]

Thanks for submitting your issue. Can you take another look at your description and make sure the issue template has been filled in its entirety?

👉 Click here if you want to take another look at the Bug Report issue template.

[ericlewis]

@schumannd it seems unused, you could try removing it.

[grabbou]

I think we should actually fix the code to handle multiple listeners. Right now, if you call addEventListener two times, the latter one will silently overwrite the former one.

I believe this is kind of behaviour you wouldn't expect.

If the API is designed to only handle single handler, it should be setEventListener.