Allow command-line deployment for Android, and making FCM-first.

[rhdeck]

Address issue #827 and #828.

1. Removes excess files not necessary for RN android modules. (IML and Proguard)
2. Add react-native-update-gradle peer dependency to allow modern build process
3. Add manifest XML file to the module so it just gets merged in
4. Update gradle files to reference the google-services.json
5. Fix code to be firebase-first (including handling onNewToken transparently via registration callback)
6. Fixing auto-deployment at react-native link via prelink script and adding rnpm reference to package.json
7. Adding support for react-native add-google-services via plugin.js and rnpm reference in package.json

Usage for testing:

react-native init mytest
yarn add rhdeck/react-native-push-notification react-native-update-gradle
react-native link
react-native add-google-services /path/to/my/google-services.json
react-native run-android

Or, if you're like me and into native dev:

yarn add react-native-android-studio
react-native studio

(Gradle synch isn't perfect in android studio right now. I sometimes have to "sync gradle with files" under the "File" menu to get things to build. This is not specific to the PR, but is general to using new gradle)

[sraka1]

@rhdeck are you using this in production? I assume this needs to be merged sometime soon anyways, since GCM is being deprecated in April...

[rhdeck]

Yes, we use this in production, and it's been smooth enough that I haven't given it a thought in a few months.

[mmccartney]

IMO this is the most import PR for this project because it adds onNewToken() and adopts the new FirebaseInstanceId API. I also appreciate the attempt to automate the changes build.gradle and AndroidManifest.xml.

[rhdeck]

What a nice note. Glad to contribute! (And to soon move off my forked version)

…

On Fri, Feb 15, 2019 at 12:25 PM Mike McCartney ***@***.***> wrote: ***@***.**** approved this pull request. IMO this is the most import PR for this project because it adds onNewToken() and adopts the new FirebaseInstanceId API. I also appreciate the attempt to automate the changes build.gradle and AndroidManifest.xml. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#836 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ALov_7IwXIF0s4BLqYfUfilhoNQKVonGks5vNu1ngaJpZM4WFowJ> .

[mmccartney]

Happy to help try to move things along. You have a conflict in package.json because the version number has already been incremented. It might help to rebase. Also, consider changing testCompile to testImplementation.

I'm going to try creating a fork that merges this and other PRs in hopes to catch up on the backlog of changes. I really want to use this project for my app but if I can't get Android working in the next few days, I'm going to have to switch to something else... :-(

[rhdeck]

Merged with current rev on my end, clearing the conflict.

[sraka1]

@rhdeck I think there are README changes needed for this as well before it can get merged... (and by changes I mean removing whole lot, since this simplifies things quite a bit)

[rhdeck]

I concur with your point. Will check it out.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 30 days if no further activity occurs. Thank you for your contributions.