feathersjs-ecosystem / feathers-redux

Integrate Feathers with your Redux store
MIT License
114 stars 23 forks source link

[docs] mention need for redux-promise-middleware #50

Closed tkrugg closed 6 years ago

tkrugg commented 6 years ago

README.md make no mention of redux-promise-middleware although it's impossible to get it working without it or an equivalent. https://github.com/feathers-plus/feathers-redux/blob/master/example/client/middleware/index.js#L12

eddyystop commented 6 years ago

A PR is welcome.

petermikitsh commented 6 years ago

ProTip: You have to include redux-promise-middleware and redux-thunk in your middleware.

https://github.com/feathers-plus/feathers-redux/blame/master/README.md#L128

It's a little low down in the README, but it's there.

tkrugg commented 6 years ago

oh, okay :) shall we close this then?