-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
[Android] Switch to Android SDK 30 & update gradle #11647
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! 👍
Have you read this https://developer.android.com/about/versions/11/behavior-changes-11? There might be some behavior changes for Androids below 11 when the app targets 11 🤔
...expo-screen-capture/android/src/main/java/expo/modules/screencapture/ScreenCapturePackage.kt
Show resolved
Hide resolved
5068fd4
to
669273d
Compare
237ff6a
to
8906142
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you make sure all the tests are green before landing? Looks like all the Android tests are failing on this PR.
177d5a8
to
127e15e
Compare
Affected packages: - @unimodules/react-native-adapter - expo-av - expo-branch - expo-error-recovery - expo-image-loader - expo-image-picker - expo-in-app-purchases - expo-media-library - expo-notifications - expo-permissions - expo-screen-capture - expo-screen-orientation - expo-splash-screen - expo-store-review - expo-updates - expo-web-browser - unimodules-permissions-interface
127e15e
to
e3b1541
Compare
# Why Followup of #11647. # How Adjust expo-media-library module according to the scoped storage documentation. > **Note:** Those changes were made with compatibility in mind. So not all changes are considered as a best practice on the newest Android. However, some of them may change the behavior of users app. # TODO - [x] Update docs # Test Plan (Tested on a simulator and real device with Android R) - NCL ✅ - test-suite ✅
# Why Followup of #11647. # How - Handled SAF files in the `expo-file-system`. - Added a way to ask for external storage write permissions. > **Note**: Those changes were needed to ensure a smooth transition to a scoped storage on Android. # Test Plan - NCL ✅
# Why We want to switch to Android SDK 30. # How - [x] I've bumped `targetSdkVersion` to `30` - [x] I've bumped `compileSdkVersion` to `30` - [x] I've fixed nullability issues coming from # Test Plan - [x] compiled & run Expo Go on Android SDK 30 simulator - [x] compiled & run `bare-expo` on Android SDK 30 simulator - [x] launched `ncl` in Expo Go and played for a while # CHANGES - [Android] Update gradle to 6.8 & gradlePlugin to 4.1.1 - [Android] Bump targerSdkVersion and compileSdkVersion to 30 - [Android] Bump kotlin plugin version to 1.4.21 - [Android] Fix & unify ANDROID_NDK_HOME path detection - [react-native][Android][submodule update] Rename ANDROID_NDK to ANDROID_NDK_HOME
Prerequisites
Why
We want to switch to Android SDK 30.
How
targetSdkVersion
to30
compileSdkVersion
to30
Test Plan
bare-expo
on Android SDK 30 simulatorncl
in Expo Go and played for a whileTODOs
CHANGELOG
entriesAfter this PR is merged
ncl
and every test intest-suite
and detect any regressions