Closed bjornevik closed 7 months ago
Android SDK/Minimum version bump seems to be in-line with the required versions in the module README.md, I'd assume that would mean that the build-properties is no longer required. The module must also upgrade to Java 17 for compatibility.
build-properties will not be needed then, but won't break if left in. As for Java 17, I don't think that needs to be updated on the module level, but just on the app/machine level. Haven't tested it out, but this module should just work with Expo 50 with no additional steps needed.
Tried a completely fresh expo project with this module earlier today (having some problems with a crash on startup for production builds, which from the testing I did doesn't seem directly related to this module).
Anyways the npx eas-cli build --platform android
command ended up failing with the following exception:
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':youssefhenna-expo-enode-link-sdk:compileReleaseKotlin'.
> 'compileReleaseJavaWithJavac' task (current target is 17) and 'compileReleaseKotlin' task (current target is 11) jvm target compatibility should be set to the same Java version.
Consider using JVM toolchain: https://kotl.in/gradle/jvm/toolchain
I did try to just edit the mentions of JavaVersion.VERSION_11
in the build.gradle directly in the node_modules directory, but didn't seem to help. Might've missed something.
I see, thanks for reporting. There's probably some work here to be done to get it to be 50-compatible. I'll try to get to it soon, or feel free to create a PR if you're up for it. We'll have to make sure these changes don't break for version 48 and 49, or we might have to release seperate versions/ maintain separate branches for different expo SDK versions
Expo SDK 50 has released
It contains the following:
Android SDK/Minimum version bump seems to be in-line with the required versions in the module README.md, I'd assume that would mean that the build-properties is no longer required. The module must also upgrade to Java 17 for compatibility.