Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved HoC details across docs #169

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

Conversation

burabure
Copy link

acdlite/flummox/#136

burabure referenced this pull request Jun 2, 2015
- Updated babel, babel-runtime, and babel-eslint
- Fixes #161
@jakedowns
Copy link

Although this hasn't been merged in yet, I was able to get the info I needed about ConnectToStores by reading: https://github.com/burabure/flummox/blob/master/docs/docs/api/higher-order-component.md

thanks for posting this

@acdlite
Copy link
Owner

acdlite commented Jun 11, 2015

@burabure Thanks for all these docs updates. 4.0.0-rc1 has been released, so very soon I'll merge these in as part of a full sweep of the docs to add/change stuff like this. Once we do we can release 4.0 proper.

@burabure
Copy link
Author

Yay!

El jue, 11 de junio de 2015 02:37 PM, Andrew Clark [email protected]
escribió:

@burabure https://github.com/burabure Thanks for all these docs
updates. 4.0.0-rc1 has been released, so very soon I'll merge these in as
part of a full sweep of the docs to add/change stuff like this. Once we do
we can release 4.0 proper.


Reply to this email directly or view it on GitHub
#169 (comment).

* the second param (props) contains the props passed to the wrapped component
*/
([postStore, userStore], props) => ({
posts: store.getPostForUser(userStore.getUserIdFromName(props.user.name))

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be postStore.getPostForUser instead of store.getPostForUser?

Nicolas added 2 commits June 18, 2015 16:10
Update higher-order-component.md
Update higher-order-component.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants