React-Native 0.29 breaking changes

[dotnetprofessional]

Plugins which pass activity reference in the constructor need to be updated to extend ReactContextBaseJavaModule use getCurrentActivity to get the activity reference. This change is backward compatible.
https://github.com/facebook/react-native/releases/tag/v0.29.0

Currently this is preventing the use of the code with the latest bits.

[axemclion]

@dotnetprofessional Thanks for reporting this. I will look at updating this plugin for that usecase.

[dotnetprofessional]

Great! Not to put any pressure on you ;) But do you have any idea when you might get the chance to update it?

[axemclion]

@dotnetprofessional It will probably be a while. I am currently working on web workers for ReactNative, that I am speaking about at conferences. I need to get those done before I work on this.

Would appreciate pull request if you have the time.

[axemclion]

@dotnetprofessional I finally got a chance to look into this and looks like that is not the only breaking change. The bigger change is the introduction of MainApplication - this is where the native modules are initialized.

This is a problem since we would not really have an Activity when the native module is initialized. Cordova's interfaces require an activity, and are always based on that. Thus, we would need to find a way to access that package in the MainActivity, initialize it when that activity is created and then run the plugin, when the exec method is called.

This looks like a total re-write of the plugin, given that so many pieces have moved around.

[avishayhajbi]

+1

[vthunder]

In case this helps anyone–

It's possible to use an older version of react-native by using the 'rninit' command:

npm install -g rninit
rninit App --source react-native@0.28.0

Then you'll be able to use react-native-cordova-plugin as-is. You'll be stuck at that version of react-native until this issue is resolved, of course, with whatever quirks and bugs that entails.

Btw, downgrading react-native and running react-native upgrade (which prompts you to downgrade files in the android/ and ios/ dirs) did not work for me. I managed to get it to build, but the app would crash on startup—there's some additional state being bundled in from somewhere that I couldn't track down. YMMV.

[sudharsan1988]

+1

[mtwallet]

@axemclion Have you had chance to look at a fix for this issue? Many thanks.

[e-onux]

@axemclion Have you had a chance to look?

[nda2221]

@axemclion Any update, friend?

[nicosefer]

any news?

[axemclion]

@nicsefer I have not had the time to work on this, but would accept pull requests.

[theBliz]

@axemclion @mribbons it seems that the pull request has some issues. The readme it's missing the MainApplication.java example and the CordovaPluginPackage.java is missing "set_MainActivity".
I don't know if something else is missing, I'll update this in case I find out more :) and thanks for the great work! :D