search

syrusakbary / react-native-refresher

A pull to refresh ListView for React Native completely written in js.
MIT License
453 stars 34 forks source link

onRefresh not called & warnings in packager #5

Open tolu360 opened 9 years ago

tolu360 commented 9 years ago

Hi,

Thanks for sharing your work. It appears that the onRefresh function is not getting called at all on pull and release - I stuck in a console.log() somewhere in the function and even that is not getting hit at all. Please what could be missing? My list seems to work normal as before, no errors there.

Except for these warnings in the packager:

Unable to resolve module precomputeStyle from /Code/MyApp/node_modules/react-native-refresher/lib/RefresherListView.js Unable to resolve module logError from /Code/MyApp/node_modules/react-native-refresher/lib/RefresherListView.js Unable to resolve module NativeMethodsMixin from /Code/MyApp/node_modules/react-native-refresher/lib/indicators/LoadingBarIndicator.js

tolu360 commented 9 years ago

My onRefresh method gets called alright now, I was doing it wrong. The warnings in packager still persist though. Thanks

syrusakbary commented 9 years ago

@tolu360 I will take a look into the warnings later ;)

zachguo commented 9 years ago

I still have these warnings and sometimes it prevents simulator from starting up.