Handle ref objects in children.

[joaovieira]

Description

Handle children refs created with React.createRef.

Motivation and context

Fixes #46

How has this been tested?

Unit test + local project.

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• My code follows the code style of this project
• I have read the CONTRIBUTING document
• I have added tests to cover my changes
• All new and existing tests passed

[Rendez]

Thank you, this is great! My concern here is about those using the typings. Should wait for those to be updated? Or inform about it in the docs so that users of this library know that it is ok to add ts-lint ignore for the time being?

[joaovieira]

There is no problem with the typings. There would be if we were trying to assign to `ref.current` in Typescript (because React types incorrectly say it's read-only). But given this module is plain JS there is no problem. For the consumer, the types don't change either. For consumers using TS (like myself) there is no problem either.

…

On Tue, 23 Oct 2018, 09:32 Luis Merino, ***@***.***> wrote: Thank you, this is great! My concern here is about those using the typings <facebook/react#13029 (comment)>. Thoughts? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#47 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAc96wJxi4h1jgb4DL7c2vN0g7lc4ckQks5untQUgaJpZM4X0B-I> .