react-native-simple-router
react-native-simple-router copied to clipboard
RN update breaks customAction
Since i have updated my project to react-native @v0.27.2 and [email protected] i get this error when i try to trigger my customAction callback
Any ideas?
Same here
Same
Same for me
fixed it by replacing
customAction(opts) {
this.customAction(opts);
}
with
customAction(opts) {
this.props.customAction(opts);
}
Where do you put this code?
In the main index of this module: yourProject/node_module/react-native-simple-router/index.js @ line 238
Other option is to get the following commit/fork:
npm i -save git+https://github.com/react-native-simple-router-community/react-native-simple-router.git#8f5ef30a093ad24f77903fa54f140260f7689798
Otherwise just submit a PR and i'll approve it
On Thu, Jul 28, 2016 at 11:28 AM Adel Bruno Grimm [email protected] wrote:
In the main index of this module: yourProject/node_module/react-native-simple-router/index.js @ line 238
Other option is to get my fork:
npm i -save git+https://github.com/react-native-simple-router-community/react-native-simple-router.git#8f5ef30a093ad24f77903fa54f140260f7689798
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/react-native-simple-router-community/react-native-simple-router/issues/140#issuecomment-235845477, or mute the thread https://github.com/notifications/unsubscribe-auth/AANjAszoOB4J8GwbfnCEz6NSDkVO4EfFks5qaHYggaJpZM4I0RCo .
Doesnt work for me. Used your commit/fork. My project is [email protected]
Well my project is also running on 29.2. I will submit a PR with my newest fix in the next ten minutes
try removing the module by hand, then run npm cache clean and than install it again and start the packager.
My bad, it breaks as soon as i try to use customAction, but i fixed it, i will submit a PR
I can see why there was a recursive loop... @charpeni mind taking a look at this as well? I don't have any RN projects to test this on anymore :(
i got it fixed and submitted a PR, should work now @michelantonides and @sethx
@BigPun86 Thank you very much. It works