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

Google play services versioning #655

Closed
wants to merge 2 commits into from

Conversation

kingbin
Copy link

@kingbin kingbin commented Feb 21, 2018

  • Gradle fails to build with other modules requiring "apply plugin: 'com.google.gms.google-services'" in app-gradle.build. ie: FCM.

The error given is:

Gradle sync failed: For input string: "+"
  • The change in this request adds the ability for a developer to pass in a version through a root project ext variable, googlePlayServicesVersion.

  • This change adds functionality to the Gradle build and is not operating system specific.

  • This change can be tested by adding FCM support to a project and then adding the apply plugin.
    app-gradle.build

...
dependencies {
    ...
    implementation project(':react-native-push-notification')
    implementation "com.google.firebase:firebase-core:$googlePlayServicesVersion"
    implementation "com.google.firebase:firebase-messaging:$googlePlayServicesVersion"
    ...
}
apply plugin: 'com.google.gms.google-services'
...

@kingbin kingbin closed this Feb 28, 2018
@kingbin kingbin deleted the googlePlayServicesVersioning branch February 28, 2018 22:13
@systemride
Copy link

Just curious why this was closed? Do you have another solution?

@kingbin kingbin restored the googlePlayServicesVersioning branch March 22, 2018 16:33
@kingbin
Copy link
Author

kingbin commented Mar 22, 2018

@systemride, I believe this request was automatically closed because I deleted my project branch. It would be awesome if this was accepted because I'm currently managing an internal fork of this project for a project I'm working on.

I wasn't really thinking that the pull request would be lost if I cleaned up my branches. I restored the branch and I'll make another pull request.

Sorry about that.

@kingbin kingbin reopened this Mar 22, 2018
@systemride
Copy link

Awesome. I'd love to see this get merged in. I am unable to build on Android cleanly without something like this.

@systemride
Copy link

Actually, looks like there is another PR with the same fix: #684

@kingbin
Copy link
Author

kingbin commented Mar 23, 2018

It'd be cool to get this in the repo and submitted to npm regardless of which pull request gets accepted.

@Gp2mv3
Copy link
Contributor

Gp2mv3 commented Jul 14, 2018

The PR #684 does exactly the same thing and is indeed merged. 😉

@Gp2mv3 Gp2mv3 closed this Jul 14, 2018
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

Successfully merging this pull request may close these issues.

3 participants