create foreground notification channel once at startup

mattermost · Oct 5, 2023 · f7c2b85 · f7c2b85
1 parent 2ecbdf5
commit f7c2b85
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/app/products/calls/connection/connection.ts b/app/products/calls/connection/connection.ts
@@ -8,7 +8,11 @@ import InCallManager from 'react-native-incall-manager';
 import {mediaDevices, MediaStream, MediaStreamTrack, RTCPeerConnection} from 'react-native-webrtc';
 
 import {setPreferredAudioRoute, setSpeakerphoneOn} from '@calls/actions/calls';
-import {foregroundServiceStart, foregroundServiceStop} from '@calls/connection/foreground_service';
+import {
+    foregroundServiceStart,
+    foregroundServiceStop,
+    foregroundServiceSetup,
+} from '@calls/connection/foreground_service';
 import {processMeanOpinionScore, setAudioDeviceInfo} from '@calls/state';
 import {AudioDevice, type AudioDeviceInfo, type AudioDeviceInfoRaw, type CallsConnection} from '@calls/types/calls';
 import {getICEServersConfigs} from '@calls/utils';
@@ -27,6 +31,11 @@ const rtcMonitorInterval = 4000;
 
 const InCallManagerEmitter = new NativeEventEmitter(NativeModules.InCallManager);
 
+// Setup the foreground service channel
+if (Platform.OS === 'android') {
+    foregroundServiceSetup();
+}
+
 export async function newConnection(
     serverUrl: string,
     channelID: string,

diff --git a/app/products/calls/connection/foreground_service.ts b/app/products/calls/connection/foreground_service.ts
@@ -14,8 +14,12 @@ const channelConfig = {
     enableVibration: false,
 };
 
+// Note: multiple calls with same arguments are a noop.
+export const foregroundServiceSetup = () => {
+    VIForegroundService.getInstance().createNotificationChannel(channelConfig);
+};
+
 export const foregroundServiceStart = async () => {
-    await VIForegroundService.getInstance().createNotificationChannel(channelConfig);
     const notificationConfig = {
         channelId: 'calls_channel',
         id: 345678,