Skip to content

Issue #383 : Error message for using refs outside of render() is difficult to underst...#716

Merged
vjeux merged 3 commits intofacebook:masterfrom
bitshadow:hotfix
Dec 28, 2013
Merged

Issue #383 : Error message for using refs outside of render() is difficult to underst...#716
vjeux merged 3 commits intofacebook:masterfrom
bitshadow:hotfix

Conversation

@bitshadow
Copy link
Contributor

...and

Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like you have an unbalanced paren here.

@vjeux
Copy link
Contributor

vjeux commented Dec 26, 2013

Thanks a lot @bitshadow, do you mind signing the CLA so we can merge it in? https://developers.facebook.com/opensource/cla Thanks!

@bitshadow
Copy link
Contributor Author

@vjeux Done.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Er, now you have an extra closing paren? :) The message for removing looks okay.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops..

@fabiomcosta
Copy link
Contributor

Seems like this is ready to be merged

vjeux added a commit that referenced this pull request Dec 28, 2013
 Issue #383 : Error message for using refs outside of render() is difficult to underst...
@vjeux vjeux merged commit 72fd246 into facebook:master Dec 28, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants