-
Notifications
You must be signed in to change notification settings - Fork 315
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
Support/android target sdk 32 #1818
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🦋 Changeset detectedLatest commit: 9d81164 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@@ -46,6 +55,7 @@ | |||
android:configChanges="keyboard|keyboardHidden|orientation|screenSize|uiMode" | |||
android:windowSoftInputMode="adjustResize" | |||
android:screenOrientation="portrait" | |||
android:exported="true" |
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.
needed for Android API >=31
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.
🌟
LGTM |
Support/android target sdk 32
Support/android target sdk 32
Support/android target sdk 32
Support/android target sdk 32
📝 Description
The Play Store now requires us to target Android SDK level >= 31.
In order to do this, we need some changes in the Bluetooth permissions flow. Here they are (I actually bumped to 32)
I only tested on Android 12 as I don't have other devices but it is necessary to test this PR with Android 9, 10 and 11 to be fully covered.
❓ Context
ledger-live-mobile
✅ Checklist
📸 Demo
🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.