-
Notifications
You must be signed in to change notification settings - Fork 29
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
React forward ref #37
Conversation
nice nice! |
<ComponentWrapper /> | ||
<div> | ||
<ComponentWrapper /> | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think ReactTestUtils.renderIntoDocument
changed a bit with react versions, so we aren't able to just have the Component
render then call unmountComponentAtNode
, i needed to wrap in a div
🤷♂️
i'd like to get a v |
…st, use object spread
@dozoisch mind giving this a final review? if good let's get a release candidate out? |
Published as the @next tag! Should be available on npm. :) |
@hartzis Amazing work, thank you so much for pushing all of this through 🎉 |
no worries! thank you for being so responsive, helpful, and available!
On Sun, Aug 12, 2018 at 18:52 Hugo Dozois ***@***.***> wrote:
@hartzis <https://github.com/hartzis> Amazing work, thank you so much for
pushing all of this through 🎉
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#37 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFISk2aUSRa6Df-vpO0RMfRsAZhRkBCnks5uQM24gaJpZM4V3NvO>
.
--
Brian Hartz
www.hartzis.me <http://www.hartzis.me/about>
brianhartz@gmail.com
|
forwardRef
getComponent
TODO:
i'd like to get a v
1.0.0-rc1
release candidate out there soonish so i can start using in codesandbox and maybe get areact-google-recaptcha
release candidate v1.0.0
out using this release candidate 😸