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

Update build.gradle #213

Merged
merged 2 commits into from
Jul 24, 2018
Merged

Update build.gradle #213

merged 2 commits into from
Jul 24, 2018

Conversation

idlework
Copy link
Contributor

Add defaults fallback when rootProject.ext does not contain sdk- and compile-versions.

the parameters compileSdkVersion, buildToolsVersion, minSdkVersion and targetSdkVersion were not available in the Root project of my project. Adding a standard fallback to the build.gradle ensures that the react-native-iap package can always compile for everyone.

idlework added 2 commits July 24, 2018 12:06
Add fallback when rootProject ext does not contain sdk- and compile-versions
@hyochan
Copy link
Owner

hyochan commented Jul 24, 2018

Thanks

@hyochan hyochan merged commit ff5ef2b into hyochan:master Jul 24, 2018
@martnst
Copy link

martnst commented Jul 26, 2018

Hey, I am wondering why it needs min API version 26. Looking at the current distribution shares that's apparently cutting of 90% of the market.

@martnst
Copy link

martnst commented Jul 26, 2018

Okay, I double checked the changes… it seems that is is now supposed to pull minSdkVersion and so on from the project's config. Unfortunately, however, it does not work for me. I guess it related to #215

FYI: I am staying at version 1.2.4 for now, which works fine fore me.

@mvanroon
Copy link
Contributor

mvanroon commented Jul 26, 2018

looks like @idlework also made a tiny mistake here

fixed in #217

@hyochan hyochan added 🤖 android Related to android upgrade labels Dec 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 android Related to android
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants