-
Notifications
You must be signed in to change notification settings - Fork 48
fetchAllData sometimes gets into loop #16
Comments
Exploring react-fetcher as an alternative on a branch. But it will also need to change when the new React Router API drops: erikras/react-redux-universal-hot-example#531 |
Hi, I've spent way to much time choosing starting point for new project and finally settled on your project. Great work! |
@tadeuszwojcik definitely near future. a hook for serverside fetching is a core piece of what universal-redux should provide, and ideally it should also allow custom serverside render hooks in case you want to do roll your own fetching logic (a kind of 'universal rendering middleware'). work on this has been blocked by the changes in updating to react-router 2.0, which are now nearly stable in the 3.x version of UR, and it's there that i intend to look into plugging in either react-fetcher or something like it. whether that makes it back to 2.x depends on if redux-simple-router can support it, which depends on the issue above. at this point i'd recommend going with UR 3.x unless there's an issue with your project code an react-router 2.0-rc4. |
I'd like to keep this issue open for mainly for backward compatibility, about returning fetchData to 2.x, and #43 for about new and potentially different ways of solving the same problem. |
cool, thanks for the update! |
@tadeuszwojcik Release 3.0.0-rc5 has async-props support. Put a static loadProps function on your views to use. https://github.com/rackt/async-props#usage |
3.0.0-rc16 now uses redux-async-connect. More info here #50 (comment) |
How do you recommend using it with: http://erikras.github.io/redux-form/#/api/reduxForm?_k=ercae5 |
@shilpan you should be able to use both the |
It has been removed for 2.0.0-beta5 pending resolution of jlongster/react-redux-universal-hot-example#4
The text was updated successfully, but these errors were encountered: