fixing mergeExtDexDebugAndroidTest task

[mynameisguy]

running mergeExtDexDebugAndroidTest task failed. fixed it as described here

PR Checklist

• The PR title follows our guidelines: https://github.com/proyecto26/react-native-inappbrowser/blob/master/CONTRIBUTING.md#pull-request-process.
• There is an issue for the bug/feature this PR is for. To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it.
• All existing tests are passing
• Tests for the changes are included

What is the current behavior?

Tried to run detox build for android in my project.
This library failed with:

> Transform artifact browser.aar (androidx.browser:browser:1.3.0-alpha01) with DexingTransform
AGPBI: {"kind":"error","text":"Static interface methods are only supported starting with Android N (--min-api 24): androidx.browser.trusted.TrustedWebActivityDisplayMode androidx.browser.trusted.TrustedWebActivityDisplayMode.fromBundle(android.os.Bundle)","sources":[{}],"tool":"D8"}

> Task :react-native-inappbrowser-reborn:mergeExtDexDebugAndroidTest FAILED

What is the new behavior?

Now I can run detox build in my root project and it doesn't fail.

[jdnichollsc]

I can't see any other package from the community using that configuration https://github.com/react-native-community, it looks like you need to include that from your project itself

[JavanHuang]

Hi all!

While configuring the detox setup for my own project, I also chanced upon this error while trying to build the detox test configurations with detox build -c <configuration-name> .

I've already included the following configuration in my own project and I'm still unable to successfully build the detox test configurations.

android {
    compileOptions {
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
    }
}

This configuration is also being used in packages from the react-native-community, and examples are react-native-community/lottie-react-native and react-native-webview

Perhaps we could try including the configuration in this package?

[jdnichollsc]

@mynameisguy can you update please the branch of this PR to develop instead of master? Thanks for your help! <3

[mynameisguy]

sure! done

[jdnichollsc]

Thanks! <3

[codetheweb]

@jdnichollsc it looks like this patch isn't published yet, do you have plans to push a new version to NPM soon?

[jdnichollsc]

@codetheweb sorry folks for the delay, let me create the new release during this next weekend! 👍

[JoaoBGusmao]

@jdnichollsc It would be nice if you could take some time to publish this change. Thanks!