Merge pull request #176 from ipanasenko/accept-thenable

Add ability to accept thenables in identity requests

README.md

@@ -328,7 +328,7 @@ connect(props => ({
 
 Note, this form of transformation is similar to what is possible on the `PromiseState` (i.e. `this.props.usersFetch.then(users => users.filter(u => u.verified))`); however, this has the advantage of only being called when `usersFetch` changes and keeps the logic out of the component.
 
-Identity requests can also be provided a `Promise`. In this case, the `PromiseState` will be `pending` until the `Promise` is resolved. This can be helpful for asynchronous, non-fetch operations (e.g. file i/o) that want to use a similar pattern as fetch operations.
+**Identity requests can also be provided a `Promise` or thenable.** In this case, the `PromiseState` will be `pending` until the `Promise` is resolved. This can be helpful for asynchronous, non-fetch operations (e.g. file i/o) that want to use a similar pattern as fetch operations.
 
 ## Accessing Headers & Metadata
 

src/components/connect.js

@@ -8,6 +8,7 @@ import PromiseState from '../PromiseState'
 import hoistStatics from 'hoist-non-react-statics'
 import invariant from 'invariant'
 import warning from 'warning'
+import hasIn from 'lodash/hasIn'
 import omit from 'lodash/fp/omit'
 
 const defaultMapPropsToRequestsToProps = () => ({})
@@ -272,7 +273,7 @@ function connect(mapPropsToRequestsToProps, defaults, options) {
         const onRejection = this.createPromiseStateOnRejection(prop, mapping, startedAt)
 
         if (mapping.hasOwnProperty('value')) {
-          if (mapping.value instanceof Promise) {
+          if (hasIn(mapping.value, 'then')) {
             this.setAtomicState(prop, startedAt, mapping, initPS(meta))
             return mapping.value.then(onFulfillment(meta), onRejection(meta))
           } else {