Always enable notifications

Previously we did disable notifications on emulators. That is not
only not needed but also counter-productive as we want to be able
to test them on different device configurations.

Removes the config flag `enableNotifications` completely and the
check that uses this flag to enable/not enable the notifications.

src/config.js

@@ -21,7 +21,6 @@ type Config = {
   enableReduxSlowReducerWarnings: boolean,
   enableSentry: boolean,
   enableWebViewErrorDisplay: boolean,
-  enableNotifications: boolean,
   slowReducersThreshold: number,
   sentryKey: string,
   enableErrorConsoleLogging: boolean,
@@ -42,7 +41,6 @@ const config: Config = {
   enableReduxSlowReducerWarnings: isDevelopment && !!global.btoa,
   enableSentry: !isDevelopment && !isEmulator,
   enableWebViewErrorDisplay: isDevelopment,
-  enableNotifications: !isEmulator,
   slowReducersThreshold: 5,
   sentryKey: 'ADD-DSN-HERE',
   enableErrorConsoleLogging: true,

src/diagnostics/VariablesScreen.js

@@ -12,7 +12,6 @@ export default class VariablesScreen extends PureComponent<{}> {
       enableReduxLogging: config.enableReduxLogging,
       enableReduxSlowReducerWarnings: config.enableReduxSlowReducerWarnings,
       enableSentry: config.enableSentry,
-      enableNotifications: config.enableNotifications,
       'process.env.NODE_ENV': process.env.NODE_ENV,
       'global.btoa': !!global.btoa,
     };

src/message/fetchActions.js

@@ -201,9 +201,7 @@ export const doInitialFetch = () => async (dispatch: Dispatch, getState: GetStat
   dispatch(fetchEssentialInitialData());
   dispatch(fetchRestOfInitialData());
 
-  if (config.enableNotifications) {
-    dispatch(initNotifications());
-  }
+  dispatch(initNotifications());
   dispatch(sendFocusPing());
   setInterval(() => sendFocusPing(), 60 * 1000);
 };