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

Fix children to be a prop in the blog post example #5695

Merged
merged 2 commits into from
Dec 21, 2015
Merged

Fix children to be a prop in the blog post example #5695

merged 2 commits into from
Dec 21, 2015

Conversation

gaearon
Copy link
Collaborator

@gaearon gaearon commented Dec 18, 2015

Whoops. Other examples seem fine.

cc @zpao

@waldreiter
Copy link
Contributor

This looks wrong to me too:

On line 12: In this traditional UI model, it is up to you to take care

On line 165 do the braces make sense? is-a ()

@gaearon
Copy link
Collaborator Author

gaearon commented Dec 18, 2015

Thanks @cody, fixed those too.

@facebook-github-bot
Copy link

@gaearon updated the pull request.

@gaearon
Copy link
Collaborator Author

gaearon commented Dec 21, 2015

ping @spicyj, can I merge this?

@sophiebits
Copy link
Collaborator

Sure. Then git cherry-pick -x on the 0.14-stable branch (or tag it with "needs merge to stable" and someone will do it… eventually).

gaearon added a commit that referenced this pull request Dec 21, 2015
Fix children to be a prop in the blog post example
@gaearon gaearon merged commit 50af034 into facebook:master Dec 21, 2015
@gaearon gaearon deleted the patch-2 branch December 21, 2015 20:04
gaearon added a commit that referenced this pull request Dec 21, 2015
Fix children to be a prop in the blog post example
(cherry picked from commit 50af034)
@gaearon
Copy link
Collaborator Author

gaearon commented Dec 21, 2015

Just did this. Should I also do this for my other PR (#5702)?

@sophiebits
Copy link
Collaborator

Just for the website. Shouldn't matter for the other one because everyone should be looking at master on github.

@gaearon
Copy link
Collaborator Author

gaearon commented Dec 21, 2015

Got it.

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.

4 participants