Переглянути джерело

fix(log): log the page reload from one place

master
paweldomas 8 роки тому
джерело
коміт
d256bc317a
2 змінених файлів з 7 додано та 5 видалено
  1. 0
    5
      conference.js
  2. 7
    0
      modules/UI/reload_overlay/PageReloadOverlay.js

+ 0
- 5
conference.js Переглянути файл

@@ -370,11 +370,6 @@ class ConferenceConnector {
370 370
 
371 371
         case ConferenceErrors.FOCUS_LEFT:
372 372
         case ConferenceErrors.VIDEOBRIDGE_NOT_AVAILABLE:
373
-            // Log the page reload event
374
-            // FIXME (CallStats - issue) this event will not make it to
375
-            // the CallStats, because the log queue is not flushed, before
376
-            // "fabric terminated" is sent to the backed
377
-            APP.conference.logEvent('page.reload');
378 373
             // FIXME the conference should be stopped by the library and not by
379 374
             // the app. Both the errors above are unrecoverable from the library
380 375
             // perspective.

+ 7
- 0
modules/UI/reload_overlay/PageReloadOverlay.js Переглянути файл

@@ -116,6 +116,13 @@ export default {
116 116
         if (!overlay) {
117 117
             overlay = new PageReloadOverlayImpl(timeoutSeconds);
118 118
         }
119
+        // Log the page reload event
120
+        if (!this.isVisible()) {
121
+            // FIXME (CallStats - issue) this event will not make it to
122
+            // the CallStats, because the log queue is not flushed, before
123
+            // "fabric terminated" is sent to the backed
124
+            APP.conference.logEvent('page.reload');
125
+        }
119 126
         overlay.show();
120 127
     }
121 128
 };

Завантаження…
Відмінити
Зберегти