Commit Graph

16 Commits

Author SHA1 Message Date
Rodrigo Troncoso c4e4306c15
fix: Bump Glide version number to v4.11.0. (#649)
Fixes #536
2020-03-12 18:48:09 -04:00
Dylan Vann 227d6e4e94 chore: Refactor Gradle variables. 2019-10-20 16:01:55 -04:00
Dylan Vann 5489f9ec4b
feat: Upgrade to React Native 0.60.0 / CocoaPods / Android X. (#513)
BREAKING CHANGE: You should upgrade React Native. See https://facebook.github.io/react-native/blog/2019/07/03/version-60
2019-07-04 23:49:38 -04:00
Dylan Vann a3b581014d Fix build.gradle. 2018-09-23 16:36:46 -04:00
Danish 649e9a8edf Update obsolete compile to implementation and Exclude app dependencies from test configurations (#250)
FIRST OF ALL, I am using this package from npm so I saw that the git repo one commit ahead that is this one f31a44fc07 (diff-7ae5a9093507568eabbf35c3b0665732) where you upgrade to glide 4 with progress listeners. I couldn't submit PR for the version I am using from NPM. so as of now explain of this PR is as follow:


As React native 0.56.0 is released with MAJOR android changes such as:

-Gradle 3.5.1
-Compile using Android SDK 26
-Google Maven repo
-etc

more about Change log, specially of Android Here, https://github.com/react-native-community/react-native-releases/blob/master/CHANGELOG.md#android-projects-are-now-compiled-using-the-android-26-sdk
and https://github.com/react-native-community/react-native-releases/blob/master/CHANGELOG.md#android-specific-changes

Aslo as Per Google new "Google Play's target API level requirement", mentioned here  https://developer.android.com/distribute/best-practices/develop/target-sdk

"Google Play will require that new apps target at least Android 8.0 (API level 26) from August 1, 2018, and that app updates target Android 8.0 from November 1, 2018."

It's clear that Android would be moving to new Gradle and new SDK changes, Thus creating a new type of error that is:

"Configuration 'compile' is obsolete and has been replaced with 'implementation' or 'Api'" 

more about the Old and new configuration please see this https://developer.android.com/studio/build/gradle-plugin-3-0-0-migration?utm_source=android-studio#new_configurations 
at the heading of "Use the new dependency configurations"

Therefore, all old "compile" should be now "implementation" or "provided" should be "compileOnly"

I have tested the with "compile" and "implementation" in my current release ready product and also by "renaming" "compile" to "implementation".
Also tested with higher andorid SDK configs such as of below:

```
ext {
    buildToolsVersion = "27.0.3"
    minSdkVersion = 19
    compileSdkVersion = 27
    targetSdkVersion = 27
    supportLibVersion = "27.1.1"
}
```

so SDK 27 was also compiled successully. test on Android phone with ADK 23 all looks good. Thus, `react-native-fast-image` was working just fine as intended. thank you so much for this package btw.

Therefore, Please would it be possible to merge as i am sure many new projects and old one are going to update specially Gradle and SDK as of Google new requirements. 

related mention of Upgrading issue mention here https://github.com/facebook/react-native/issues/20273#issuecomment-405959030


Moreover, if someone still have any issue after upgrading everything successfully, then he should use following in build.gradle under `andorid/build.gradle`

```

subprojects {
    afterEvaluate {project ->
        if (project.hasProperty("android")) {
            android {
                compileSdkVersion 27
                buildToolsVersion '27.0.3'
                supportLibVersion = "27.1.1"
            }
        }
    }
}
```


PS: as of exclude part in your build.grade, I am not 100% it will work for example mine app has something like this

compile project((':react-native-maps')) {
        exclude group: 'com.google.android.gms', module: 'play-services-base'
        exclude group: 'com.google.android.gms', module: 'play-services-maps'
}
compile 'com.google.android.gms:play-services-base:10.0.1'
compile 'com.google.android.gms:play-services-maps:10.0.1'


after converting this to


implementation project(':react-native-maps')
 android.testVariants.all { variant ->
        variant.getCompileConfiguration().exclude group: 'com.google.android.gms', module: 'play-services-base'
        variant.getRuntimeConfiguration().exclude group: 'com.google.android.gms', module: 'play-services-base'

        variant.getCompileConfiguration().exclude group: 'com.google.android.gms', module: 'play-services-maps'
        variant.getCompileConfiguration().exclude group: 'com.google.android.gms', module: 'play-services-maps'
}
implementation 'com.google.android.gms:play-services-base:10.0.1'
implementation 'com.google.android.gms:play-services-maps:10.0.1'


everything is working fine for me. so please have a look for your just in case.
2018-09-23 16:33:31 -04:00
Danish 5d973a8cec Update obsolete compile to implementation and Exclude app dependencies from test configurations.
FIRST OF ALL, I am using this package from npm so I saw that the git repo one commit ahead that is this one f31a44fc07 (diff-7ae5a9093507568eabbf35c3b0665732) where you upgrade to glide 4 with progress listeners. I couldn't submit PR for the version I am using from NPM. so as of now explain of this PR is as follow:

As React native 0.56.0 is released with MAJOR android changes such as:

-Gradle 3.5.1
-Compile using Android SDK 26
-Google Maven repo
-etc

more about Change log, specially of Android Here, https://github.com/react-native-community/react-native-releases/blob/master/CHANGELOG.md#android-projects-are-now-compiled-using-the-android-26-sdk
and https://github.com/react-native-community/react-native-releases/blob/master/CHANGELOG.md#android-specific-changes

Aslo as Per Google new "Google Play's target API level requirement", mentioned here  https://developer.android.com/distribute/best-practices/develop/target-sdk

"Google Play will require that new apps target at least Android 8.0 (API level 26) from August 1, 2018, and that app updates target Android 8.0 from November 1, 2018."

It's clear that Android would be moving to new Gradle and new SDK changes, Thus creating a new type of error that is:

"Configuration 'compile' is obsolete and has been replaced with 'implementation' or 'Api'"

more about the Old and new configuration please see this https://developer.android.com/studio/build/gradle-plugin-3-0-0-migration?utm_source=android-studio#new_configurations
at the heading of "Use the new dependency configurations"

Therefore, all old "compile" should be now "implementation" or "provided" should be "compileOnly"

I have tested the with "compile" and "implementation" in my current release ready product and also by "renaming" "compile" to "implementation".
Also tested with higher andorid SDK configs such as of below:

```
ext {
    buildToolsVersion = "27.0.3"
    minSdkVersion = 19
    compileSdkVersion = 27
    targetSdkVersion = 27
    supportLibVersion = "27.1.1"
}
```

so SDK 27 was also compiled successully. test on Android phone with ADK 23 all looks good. Thus, `react-native-fast-image` was working just fine as intended. thank you so much for this package btw.

Therefore, Please would it be possible to merge as i am sure many new projects and old one are going to update specially Gradle and SDK as of Google new requirements.

related mention of Upgrading issue mention here https://github.com/facebook/react-native/issues/20273#issuecomment-405959030

Moreover, if someone still have any issue after upgrading everything successfully, then he should use following in build.gradle under `andorid/build.gradle`

```

subprojects {
    afterEvaluate {project ->
        if (project.hasProperty("android")) {
            android {
                compileSdkVersion 27
                buildToolsVersion '27.0.3'
                supportLibVersion = "27.1.1"
            }
        }
    }
}
```

PS: as of exclude part in your build.grade, I am not 100% it will work for example mine app has something like this

compile project((':react-native-maps')) {
        exclude group: 'com.google.android.gms', module: 'play-services-base'
        exclude group: 'com.google.android.gms', module: 'play-services-maps'
}
compile 'com.google.android.gms:play-services-base:10.0.1'
compile 'com.google.android.gms:play-services-maps:10.0.1'

after converting this to

implementation project(':react-native-maps')
 android.testVariants.all { variant ->
        variant.getCompileConfiguration().exclude group: 'com.google.android.gms', module: 'play-services-base'
        variant.getRuntimeConfiguration().exclude group: 'com.google.android.gms', module: 'play-services-base'

        variant.getCompileConfiguration().exclude group: 'com.google.android.gms', module: 'play-services-maps'
        variant.getCompileConfiguration().exclude group: 'com.google.android.gms', module: 'play-services-maps'
}
implementation 'com.google.android.gms:play-services-base:10.0.1'
implementation 'com.google.android.gms:play-services-maps:10.0.1'

everything is working fine for me. so please have a look for your just in case.
2018-09-23 16:28:34 -04:00
Dylan Vann b504131083 Use supportLibVersion from ext. 2018-08-23 13:43:27 -04:00
Dylan Vann 93b59295ce Fix android build.gradle. 2018-08-23 13:36:25 -04:00
Dylan Vann f31a44fc07
Upgrade to glide 4 with progress listeners working. (#201) 2018-05-10 02:40:04 -04:00
Dylan Vann a4e6ef3002 Revert increasing buildToolsVersion and targetSdkVersion. 2018-05-06 03:47:33 -04:00
Dylan Vann bf67634a44 Fix android project. 2018-05-06 03:39:22 -04:00
Rayron Victor bbdfd33aaf Using SDK Version variables from root project
Instead of assuming the `compileSdkVersion`, `targetSdkVersion`, etc, read it from the root project.
Default `compileSdkVersion` and `targetSdkVersion` to the latest versions.

Android Target API Level 26 will be required in August 2018.
https://android-developers.googleblog.com/2017/12/improving-app-security-and-performance.html
And the React Native team is already working on this:
facebook/react-native#17741
facebook/react-native#18095
2018-04-24 14:56:37 -03:00
Dylan Vann 71c5e63aee
Revert "Upgrade android to Glide v4 " 2018-03-18 02:22:12 -04:00
Kingsley Ochu 09b46283ea Upgrade glide, okhtt3 to 4.3.1 2017-11-11 11:42:49 +01:00
Dylan Vann c643347d4c Add android progress callback. 2017-07-05 04:10:21 -04:00
Dylan Vann 1ee6d37bd4 🚩 Initial commit. 2017-04-17 18:42:16 -04:00