fix: EventEmitter warnings

[ferdicus]

Description

Native EventEmitters api seems to have slightly changed.
They expect a addListener removeListener method on the native modules.

Would be showing warning messages like these:
'new NativeEventEmitter() was called with a non-null argument without the required addListener method.'

See: facebook/react-native@114be1d

Additionally, EventEmitter.js itself does change removeListener => just remove, which is called on the return value of addListener.
I've changed the code accordingly.

Would be showing warning messages like these:
EventEmitter.removeListener(): Method has been deprecated. Please instead use remove() on the subscription returned by EventEmitter.addListener.

What I didn't do where extensive test on those components/ files.
I've did some simple playing around within the example app - would appreciate some full fledged tests with this PR though

Fixes #1594

Checklist

• I have tested this on a device/simulator for each compatible OS
• I formatted JS and TS files with yarn lint
• I mentioned this change in CHANGELOG.md

[lisabaut]

Hi Ferdi, I assume the mock needs to be changed too:
https://github.com/react-native-mapbox-gl/maps/blob/master/__tests__/__mocks__/react-native.mock.js#L8

[ferdicus]

Hi Ferdi, I assume the mock needs to be changed too: https://github.com/react-native-mapbox-gl/maps/blob/master/__tests__/__mocks__/react-native.mock.js#L8

you are correct - thank you 👍🏿