jdolle / props-expedition Goto Github PK

The solution may be related to #1. Another argument could be added. Another option may be to add a class method isLoading(key: string) and handle the state internally.

I propose to wrap component render function to provide props as the first argument and errors as the second.

Wrap SFCs to do the same.

By solving the problem like this, the method of rendering components is unified. No more this.props.

However, by solving it this way we also add a little "magic" on top of React. An alternative might be to provide this.errors for components.

Travis CI

What matters to Redux is the reduced state not the result of the dispatch.
That said, using @memoize() with a dispatch makes sense.

That said, it would technically be possible by creating a 'pointless map'. I.e.

@memoize(nextProps => nextProps.id)
userDispatch: (nextProps) => nextProps.dispatch({ type: 'USER', id: nextProps.id })
user: (nextProps) => nextProps.userFromRedux // <- this is possible but semi pointless because mapStateToProps does mapping already

Add API documentation to the README.md

Definitely need to document the API/usage with Typescript in mind as well.