this fixes issue: prevent to set promise to the ui #51

rodik · rodik · commit f5bee87204a0 · 2016-03-04T10:29:57.000+02:00
diff --git a/modules/asyncConnect.js b/modules/asyncConnect.js
@@ -118,9 +118,10 @@ function wrapWithDispatch(asyncItems) {
           dispatch(load(item.key));
           promiseOrResult.then(data => dispatch(loadSuccess(item.key, data)))
                          .catch(err => dispatch(loadFail(item.key, err)));
+        } else {
+          dispatch(loadSuccess(item.key, promiseOrResult));
         }
 
-        dispatch(loadSuccess(item.key, promiseOrResult));
       }
       return promiseOrResult;
     }} : item

Original file line number	Diff line number	Diff line change
`@@ -118,9 +118,10 @@ function wrapWithDispatch(asyncItems) {`
`118`	`118`	`dispatch(load(item.key));`
`119`	`119`	`promiseOrResult.then(data => dispatch(loadSuccess(item.key, data)))`
`120`	`120`	`.catch(err => dispatch(loadFail(item.key, err)));`
	`121`	`+ } else {`
	`122`	`+ dispatch(loadSuccess(item.key, promiseOrResult));`
`121`	`123`	`}`
`122`	`124`
`123`		`- dispatch(loadSuccess(item.key, promiseOrResult));`
`124`	`125`	`}`
`125`	`126`	`return promiseOrResult;`
`126`	`127`	`}} : item`