dep(@notifee/react-native): registerForegroundService as early as pos…

…sible

index.android.js

@@ -1,2 +1,26 @@
 import './react/index.native';
+import notifee, {EventType} from '@notifee/react-native';
+import logger from './react/features/app/logger';
 
+// Needs to be registered outside any React components as early as possible
+await notifee.registerForegroundService(() => {
+    return new Promise(resolve => {
+        logger.warn('Foreground service running');
+
+        notifee.onBackgroundEvent(async event => {
+            if (event.type === EventType.ACTION_PRESS && event.detail.pressAction.id === 'hang-up') {
+                console.log('HANG UP BUTTON PRESSED BACKGROUND');
+            }
+
+            return resolve();
+        });
+
+        notifee.onForegroundEvent(async ({ type, detail }) => {
+            if (type === EventType.ACTION_PRESS && detail.pressAction.id === 'hang-up') {
+                console.log('HANG UP BUTTON PRESSED');
+            }
+
+            return resolve();
+        });
+    });
+});

react/features/conference/components/functions.native.ts

@@ -1,5 +1,5 @@
 import { IReduxState } from '../../app/types';
-import notifee, { AndroidImportance, AndroidVisibility, AuthorizationStatus } from '@notifee/react-native';
+import notifee, {AndroidImportance, AndroidVisibility, AuthorizationStatus, EventType} from '@notifee/react-native';
 import logger from "../../app/logger";
 
 export * from './functions.any';
@@ -35,14 +35,7 @@ export const isConnecting = (state: IReduxState) => {
 
 export async function displayNotificationAsForegroundService() {
 
-    // Foreground service needs to be registered first
-    notifee.registerForegroundService(() => {
-        return new Promise(() => {
-            logger.warn('Foreground service running');
-        });
-    });
-
-    // Request permissions
+    // Request permissions if not authorized
     const currentPermissions = await notifee.getNotificationSettings();
 
     if (currentPermissions.authorizationStatus !== AuthorizationStatus.AUTHORIZED) {
@@ -56,27 +49,24 @@ export async function displayNotificationAsForegroundService() {
         visibility: AndroidVisibility.PUBLIC
     });
 
-    // Display notification
+    // Display notification as a foreground service
     await notifee.displayNotification({
-        title: '<p style="color: #4caf50;"><b>Ongoing conference</span></p></b></p> &#128576;',
-        subtitle: '&#129395;',
+        title: 'Ongoing conference',
         body:
-            'Meeting <p style="text-decoration: line-through">you are </p> participating <p style="color: #ffffff; background-color: #9c27b0"><i>is ongoing</i></p> &#127881;!',
+            'Meeting you are participating is ongoing.',
         android: {
             asForegroundService: true,
             autoCancel: false,
             channelId,
             importance: AndroidImportance.HIGH,
-            color: '#4caf50',
             ongoing: true,
+            smallIcon: 'ic_notification',
             actions: [
                 {
-                    title: '<b>Dance</b> &#128111;',
-                    pressAction: { id: 'dance' },
-                },
-                {
-                    title: '<p style="color: #f44336;"><b>Cry</b> &#128557;</p>',
-                    pressAction: { id: 'cry' },
+                    title: 'Hang Up',
+                    pressAction: {
+                        id: 'hang-up'
+                    },
                 },
             ],
         },
@@ -86,7 +76,8 @@ export async function displayNotificationAsForegroundService() {
 export async function stopForegroundService() {
     try {
         await notifee.stopForegroundService();
+        logger.warn('Foreground service stopped.')
     } catch (err) {
-        console.log(err);
+        logger.error(err);
     }
 }

react/features/conference/components/native/Conference.tsx

@@ -213,8 +213,6 @@ class Conference extends AbstractConference<IProps, State> {
 
         BackHandler.addEventListener('hardwareBackPress', this._onHardwareBackPress);
 
-        displayNotificationAsForegroundService();
-
         if (_audioOnlyEnabled && _startCarMode) {
             navigation.navigate(screen.conference.carmode);
         }
@@ -232,6 +230,10 @@ class Conference extends AbstractConference<IProps, State> {
             _startCarMode
         } = this.props;
 
+        if (Platform.OS === 'android') {
+            displayNotificationAsForegroundService();
+        }
+
         if (!prevProps._showLobby && _showLobby) {
             navigate(screen.lobby.root);
         }
@@ -254,11 +256,14 @@ class Conference extends AbstractConference<IProps, State> {
      * @returns {void}
      */
     componentWillUnmount() {
+
+        if (Platform.OS === 'android') {
+            stopForegroundService();
+        }
+
         // Tear handling any hardware button presses for back navigation down.
         BackHandler.removeEventListener('hardwareBackPress', this._onHardwareBackPress);
 
-        stopForegroundService();
-
         clearTimeout(this._expandedLabelTimeout.current ?? 0);
     }