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

@expo/config-plugins and expo-module-scripts packages updated #11

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

bilal1031
Copy link

@bilal1031 bilal1031 commented Feb 14, 2024

This pull request updates the react-native-imglysdk package and includes updates to @expo/config-plugins and expo-module-scripts. The rationale behind these updates is to address compatibility issues with Expo SDK 49 which is currently supported in https://github.com/imgly/pesdk-react-native/releases/tag/3.1.0 and https://github.com/imgly/vesdk-react-native/releases/tag/3.1.0

Cause for changes:
I tried making a development build for Android in Expo to test Img.ly SDKs. However, the build crashed, and upon debugging, I found that the package version used in react-native-imglysdk is outdated and incompatible with Expo SDK 49.

The main changes include:
react-native-imglysdk Update: Incorporating the latest changes to ensure compatibility and leverage new features or improvements.
@expo/config-plugins Update: Updating to version ~7.2.0, which is essential for Expo SDK 49 compatibility. Older versions were causing build errors.
expo-module-scripts Update: Ensuring the build scripts align with the updated dependencies, enhancing the overall build process and stability.

These updates are critical to resolve issues encountered during the build process and ensure that developers can continue to work seamlessly with the latest dependencies. Kindly take a look into the PR and if you find any issues, let me know.

Screenshots of issue:
Expo doctor that gives warning about incompatible packages

image

Gradle Error:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant