Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RN update breaks customAction #140

Open
BigPun86 opened this issue Jun 13, 2016 · 14 comments
Open

RN update breaks customAction #140

BigPun86 opened this issue Jun 13, 2016 · 14 comments

Comments

@BigPun86
Copy link
Contributor

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

screen shot 2016-06-13 at 14 10 35

Any ideas?

@maluramichael
Copy link
Contributor

Same here

@maybewaityou
Copy link

Same

@Antonides
Copy link

Same for me

@BigPun86
Copy link
Contributor Author

BigPun86 commented Jul 28, 2016

fixed it by replacing

customAction(opts) {
    this.customAction(opts);
 }

with

customAction(opts) {
   this.props.customAction(opts);
 }

@Antonides
Copy link

Where do you put this code?

@BigPun86
Copy link
Contributor Author

BigPun86 commented Jul 28, 2016

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

@davidLeonardi
Copy link
Contributor

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
#140 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANjAszoOB4J8GwbfnCEz6NSDkVO4EfFks5qaHYggaJpZM4I0RCo
.

@Antonides
Copy link

Antonides commented Jul 28, 2016

Doesnt work for me. Used your commit/fork. My project is [email protected]

@BigPun86
Copy link
Contributor Author

Well my project is also running on 29.2. I will submit a PR with my newest fix in the next ten minutes

@BigPun86
Copy link
Contributor Author

try removing the module by hand, then run npm cache clean and than install it again and start the packager.

@BigPun86
Copy link
Contributor Author

BigPun86 commented Jul 28, 2016

My bad, it breaks as soon as i try to use customAction, but i fixed it, i will submit a PR

@davidLeonardi
Copy link
Contributor

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 :(

@BigPun86
Copy link
Contributor Author

BigPun86 commented Jul 28, 2016

i got it fixed and submitted a PR, should work now @michelantonides and @SEthX

@Antonides
Copy link

@BigPun86 Thank you very much. It works

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants