meteorrn / meteor-react-native

Meteor client for React Native matching Meteor Spec
https://guide.meteor.com/react-native.html
Other
59 stars 31 forks source link

Provide useTracker function #17

Closed sandeepjain closed 4 years ago

TheRealNate commented 4 years ago

Hi @sandeepjain, thanks for the PR! Have you been able to test this in production?

TheRealNate commented 4 years ago

Hi @sandeepjain, the new CONTRIBUTING.md file contains our community guidelines for contributions to MeteorRN. Have you been able to test this PR against the test cases mentioned?

TheRealNate commented 4 years ago

Merging this into the dev branch so further testing can take place.

lgtm-com[bot] commented 4 years ago

This pull request introduces 1 alert when merging ca227cc7e9e3c796c50e8bcb10ad0601f4661b25 into 6064ccf846f305fb3b9e50c7983ce0902ec1729d - view on LGTM.com

new alerts:

sandeepjain commented 4 years ago

@TheRealNate Apologies for the absence all this while as I had other priorities to take care of.

I have had a preliminary run through and this function isn't working as expected. I will try to find some time to work on the fix. In meanwhile, if someone is willing to help it would be good as well.

TheRealNate commented 4 years ago

Hi @sandeepjain, I've opened a new issue at #29 for tracking purposes.