fix: feature acknowlegement

web-pal · Mar 14, 2018 · 1721ffb · 1721ffb
1 parent ff76777
commit 1721ffb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/app/components/FeatureHighlight/FeatureHighlight.jsx b/app/components/FeatureHighlight/FeatureHighlight.jsx
@@ -29,9 +29,10 @@ const FeatureHighlight = ({
   ? (
     <Tooltip
       content={description}
-      onClick={() => dispatch(uiActions.acknowlegdeFeature({ featureId: 'multiAccounts' }))}
     >
-      <FeatureContainer>
+      <FeatureContainer
+        onClick={() => dispatch(uiActions.acknowlegdeFeature({ featureId: 'multiAccounts' }))}
+      >
         {children}
         <Lozenge appearance="new">New</Lozenge>
       </FeatureContainer>

diff --git a/app/sagas/ui.js b/app/sagas/ui.js
@@ -7,7 +7,6 @@ import {
   takeEvery,
   put,
   call,
-  take,
   fork,
   select,
 } from 'redux-saga/effects';
@@ -223,12 +222,13 @@ export function* watchSetIssuesFilter(): Generator<*, *, *> {
 }
 
 export function* newFeaturesFlow(): Generator<*, *, *> {
-  while (true) {
-    const { payload: { featureId } } = yield take(actionTypes.ACKNOWLEDGE_FEATURE);
+  function* flow({ payload: { featureId } }): Generator<*, void, *> {
     let acknowlegdedFeatures = yield call(getFromStorage, 'acknowlegdedFeatures');
     if (!acknowlegdedFeatures) acknowlegdedFeatures = [];
     acknowlegdedFeatures.push(featureId);
     yield call(setToStorage, 'acknowlegdedFeatures', acknowlegdedFeatures);
+    yield call(delay, 10000);
     yield put(uiActions.setUiState('acknowlegdedFeatures', acknowlegdedFeatures));
   }
+  yield takeLatest(actionTypes.ACKNOWLEDGE_FEATURE, flow);
 }