Skip to content

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
Contributor

@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