Merge pull request #2666 from daily-co/vp-eng-1234-fix-record-being-s…

…et-to-true-for-remote Vp eng 1234 fix record being set to true for remote
daily-co · Feb 25, 2021 · c92a211 · c92a211
2 parents bceee20 + d1174d6
commit c92a211
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/module.js b/src/module.js
@@ -2135,7 +2135,7 @@ export default class DailyIframe extends EventEmitter {
     if (!prevP) {
       return;
     }
-    if (thisP[key] === false && prevP[key] !== thisP[key]) {
+    if (!thisP.local && thisP[key] === false && prevP[key] !== thisP[key]) {
       try {
         this.emit(DAILY_EVENT_RECORDING_STOPPED, {
           action: DAILY_EVENT_RECORDING_STOPPED,
@@ -2151,7 +2151,7 @@ export default class DailyIframe extends EventEmitter {
     if (!prevP) {
       return;
     }
-    if (thisP[key] === true && prevP[key] !== thisP[key]) {
+    if (!thisP.local && thisP[key] === true && prevP[key] !== thisP[key]) {
       try {
         this.emit(DAILY_EVENT_RECORDING_STARTED, {
           action: DAILY_EVENT_RECORDING_STARTED,