fix(tracking): don't try to close UI on a not triggered context if no…

…t opened
dis-moi · Nov 22, 2019 · 2d1ed1b · 2d1ed1b
1 parent 30a9a5e
commit 2d1ed1b
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/app/content/sagas/notices.ts b/src/app/content/sagas/notices.ts
@@ -6,7 +6,7 @@ import { CLOSED } from 'app/constants/ActionTypes';
 import { StatefulNotice } from 'app/lmem/notice';
 import { CloseCause } from 'app/lmem/ui';
 import { AppAction, createErrorAction } from 'app/actions';
-import { getNotices, hasNoticesToDisplay } from '../selectors';
+import { getNotices, hasNoticesToDisplay, isOpen } from '../selectors';
 
 export function* closeIfNoMoreNoticeToDisplaySaga() {
   try {
@@ -47,7 +47,10 @@ export const isChangeOnNoticeAction = (action: AppAction) =>
   action.type === 'MARK_NOTICE_READ' || action.type === 'FEEDBACK_ON_NOTICE';
 
 function* closeUISaga() {
-  yield put(close(CloseCause.NoMoreNotice));
+  const open = yield select(isOpen);
+  if (open) {
+    yield put(close(CloseCause.NoMoreNotice));
+  }
 }
 
 export default function* noticesRootSaga() {