react-native-simple-router icon indicating copy to clipboard operation
react-native-simple-router copied to clipboard

RN update breaks customAction

Open BigPun86 opened this issue 8 years ago • 14 comments

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?

BigPun86 avatar Jun 13 '16 12:06 BigPun86

Same here

maluramichael avatar Jul 22 '16 08:07 maluramichael

Same

maybewaityou avatar Jul 27 '16 08:07 maybewaityou

Same for me

Antonides avatar Jul 28 '16 08:07 Antonides

fixed it by replacing

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

with

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

BigPun86 avatar Jul 28 '16 09:07 BigPun86

Where do you put this code?

Antonides avatar Jul 28 '16 09:07 Antonides

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

BigPun86 avatar Jul 28 '16 09:07 BigPun86

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 .

davidLeonardi avatar Jul 28 '16 09:07 davidLeonardi

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

Antonides avatar Jul 28 '16 09:07 Antonides

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

BigPun86 avatar Jul 28 '16 09:07 BigPun86

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

BigPun86 avatar Jul 28 '16 09:07 BigPun86

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

BigPun86 avatar Jul 28 '16 09:07 BigPun86

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

davidLeonardi avatar Jul 28 '16 10:07 davidLeonardi

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

BigPun86 avatar Jul 28 '16 10:07 BigPun86

@BigPun86 Thank you very much. It works

Antonides avatar Jul 28 '16 11:07 Antonides