plaid / react-native-plaid-link-sdk

Plaid Link for React Native
MIT License
171 stars 120 forks source link

work-runtime-ktx-2.7.1-runtime causing build error with latest Datadog sdk on android #660

Closed jesse-one-app closed 1 month ago

jesse-one-app commented 4 months ago

Is your feature request related to a problem? Please describe. The plaid android is using an old version of the android work runtime which has a class collision with other dependencies that have upgraded to the 2.8+ version of android work.

Describe the solution you'd like Would appreciate if plaid makes the upgrades internally to the work sdk version they use and adopt the non ktx version

Describe alternatives you've considered I dont think there is any alternative. I am blocked from upgrading datadog until Plaid makes these changes

AlexLFundbox commented 2 months ago

Any updates in this one?

melissaosullivan commented 2 months ago

Hi @jesse-one-app, we can make an upgrade to this library. Upgrading to androidx.work:work-runtime-ktx:2.9.0 would resolve this issue, right?

jesse-one-app commented 2 months ago

I believe it would yes

I was actually able to work around this by overriding the version used in my own gradle file but I dont know if there are any adverse side effects from doing that

melissaosullivan commented 2 months ago

Great! Thanks, we will put this in the next release.

AlexLFundbox commented 2 months ago

@jesse-one-app Is it possible also to share your work around ?

jesse-one-app commented 2 months ago

@AlexLFundbox sure

i added this line to my app/build.gradle to force the work runtime version

implementation "androidx.work:work-runtime-ktx:2.8.1"

AlexLFundbox commented 2 months ago

@jesse-one-app I added it to my "app/build.gradle"

but now, I'm receiving this error

`> Task :datadog_mobile-react-native:runKtlintCheckOverMainSourceSet FAILED

Task :app:minifyReleaseWithR8 WARNING: R8: Class com.facebook.imagepipeline.image.CloseableAnimatedImage extends com.facebook.imagepipeline.image.CloseableImage which is an interface

FAILURE: Build failed with an exception.

Any suggestions?

jesse-one-app commented 2 months ago

I'm not sure sorry, didn't run into this with my setup

dtroupe-plaid commented 1 month ago

11.12.0 resolves this.