Adds a method to ReactMeteorDataComponent for retrieving the wrapped … #245
Conversation
|
@rooch84: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Meteor Contributor Agreement here: https://contribute.meteor.com/ |
| getMeteorData() { | ||
| return getMeteorData(this.props); | ||
| } | ||
| getWrappedComponent() { |
There was a problem hiding this comment.
Would you mind fixing the spacing here to align with the style used in the rest of the file?
|
This would be fixed by #266 I guess. |
|
Chris Rooney seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
1 similar comment
|
Chris Rooney seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
|
On version 2.0.0 a lot of things were changed then if you think this PR is still valid please update it and open again. Thanks. |
|
Has this been implemented? |
|
@derwaldgeist in the current release, withTracker() uses React.forwardRef, so you should be able to get the wrapped component symply by using a regular React ref |
|
@yched Thanks for your feedback. Yeah, I realized this in the meantime. I was still using version 1.0.0 of this package so far, where it did not work. Had to update Meteor to 1.10 though, because of version dependencies on the ecmascript package. |
7 participants
…component.
This is useful if the parent component needs to call a method on the child.