feat(MeetingsSdkAdapter): add browser default speaker to the availabl…

…e speaker list
webex · Nov 19, 2021 · e149789 · e149789
1 parent 6d1d84a
commit e149789
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 2 deletions.
diff --git a/src/MeetingsSDKAdapter.js b/src/MeetingsSDKAdapter.js
@@ -521,7 +521,7 @@ export default class MeetingsSDKAdapter extends MeetingsAdapter {
         state: MeetingState.NOT_JOINED,
         cameraID: null,
         microphoneID: null,
-        speakerID: null,
+        speakerID: '',
       })),
       // if not provided by the sdk, compute a meeting title
       concatMap((meeting) => (

diff --git a/src/MeetingsSDKAdapter.test.js b/src/MeetingsSDKAdapter.test.js
@@ -404,7 +404,7 @@ describe('Meetings SDK Adapter', () => {
           state: 'NOT_JOINED',
           cameraID: null,
           microphoneID: null,
-          speakerID: null,
+          speakerID: '',
         });
         done();
       });

diff --git a/src/MeetingsSDKAdapter/controls/SwitchSpeakerControl.js b/src/MeetingsSDKAdapter/controls/SwitchSpeakerControl.js
@@ -43,6 +43,7 @@ export default class SwitchSpeakerControl extends MeetingControl {
         value: speaker.deviceId,
         label: speaker.label,
       }))),
+      map((options) => ([{value: '', label: 'Browser Default'}, ...options])),
     );
 
     return combineLatestImmediate(speakerID$, options$).pipe(

diff --git a/src/MeetingsSDKAdapter/controls/SwitchSpeakerControl.test.js b/src/MeetingsSDKAdapter/controls/SwitchSpeakerControl.test.js
@@ -21,6 +21,10 @@ describe('Switch Speaker Control', () => {
             tooltip: 'Speaker Devices',
             noOptionsMessage: 'No available speakers',
             options: [
+              {
+                value: '',
+                label: 'Browser Default',
+              },
               {
                 value: 'default',
                 label: 'Default - Headset Earphone (Jabra EVOLVE 20 SE MS) (0b0e:0300)',