Simplify notification subscription (#1848)

src/boot/AppEventHandlers.js

@@ -3,12 +3,16 @@ import React, { PureComponent } from 'react';
 import { AppState, NetInfo, View, StyleSheet, Platform, NativeModules } from 'react-native';
 import SafeArea from 'react-native-safe-area';
 import Orientation from 'react-native-orientation';
-import NotificationsIOS, { NotificationsAndroid } from 'react-native-notifications';
 
 import type { Auth, Actions, ChildrenArray } from '../types';
 import connectWithActions from '../connectWithActions';
 import { getAuth, getUnreadByHuddlesMentionsAndPMs } from '../selectors';
-import { handlePendingNotifications, tryInitialNotification } from '../utils/notifications';
+import {
+  addNotificationListener,
+  removeNotificationListener,
+  handlePendingNotifications,
+  tryInitialNotification,
+} from '../utils/notifications';
 
 const componentStyles = StyleSheet.create({
   wrapper: {
@@ -52,7 +56,7 @@ class AppEventHandlers extends PureComponent<Props> {
     }
   };
 
-  handleNotificationOpen = notification => {
+  handleNotificationOpen = (notification: Object) => {
     const { actions } = this.props;
     actions.saveInitialNotificationDetails(notification.getData());
     setTimeout(() => handlePendingNotifications(notification), 600);
@@ -71,21 +75,15 @@ class AppEventHandlers extends PureComponent<Props> {
     AppState.addEventListener('memoryWarning', this.handleMemoryWarning);
     SafeArea.getSafeAreaInsetsForRootView().then(actions.initSafeAreaInsets);
     Orientation.addOrientationListener(this.handleOrientationChange);
-    if (Platform.OS === 'ios') {
-      NotificationsIOS.addEventListener('notificationOpened', this.handleNotificationOpen);
-    } else {
-      NotificationsAndroid.setNotificationOpenedListener(this.handleNotificationOpen);
-    }
+    addNotificationListener(this.handleNotificationOpen);
   }
 
   componentWillUnmount() {
     NetInfo.removeEventListener('connectionChange', this.handleConnectivityChange);
     AppState.removeEventListener('change', this.handleAppStateChange);
     AppState.removeEventListener('memoryWarning', this.handleMemoryWarning);
     Orientation.removeOrientationListener(this.handleOrientationChange);
-    if (Platform.OS === 'ios') {
-      NotificationsIOS.removeEventListener('notificationOpened', this.handleNotificationOpen);
-    }
+    removeNotificationListener(this.handleNotificationOpen);
   }
 
   render() {

src/utils/notifications.android.js

@@ -27,6 +27,14 @@ const handleRegistrationUpdates = (auth: Auth, saveTokenPush: Actions.saveTokenP
   });
 };
 
+export const addNotificationListener = (notificationHandler: (notification: Object) => void) => {
+  NotificationsAndroid.setNotificationOpenedListener(notificationHandler);
+};
+
+export const removeNotificationListener = (
+  notificationHandler: (notification: Object) => void,
+) => {};
+
 export const initializeNotifications = (auth: Auth, saveTokenPush: Actions.saveTokenPush) => {
   handleRegistrationUpdates(auth, saveTokenPush);
 };

src/utils/notifications.ios.js

@@ -20,6 +20,14 @@ const onPushRegistrationFailed = (error: string) => {
   logErrorRemotely(new Error(error), 'register ios push token failed');
 };
 
+export const addNotificationListener = (notificationHandler: (notification: Object) => void) => {
+  NotificationsIOS.addEventListener('notificationOpened', notificationHandler);
+};
+
+export const removeNotificationListener = (notificationHandler: (notification: Object) => void) => {
+  NotificationsIOS.removeEventListener('notificationOpened', notificationHandler);
+};
+
 export const initializeNotifications = (auth: Auth, saveTokenPush: Actions.saveTokenPush) => {
   NotificationsIOS.addEventListener('remoteNotificationsRegistered', deviceToken =>
     onPushRegistered(auth, deviceToken, saveTokenPush),