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

Support for Android 13 and small corrections #37

Merged
merged 1 commit into from
Dec 19, 2023

Conversation

nsknyc
Copy link
Contributor

@nsknyc nsknyc commented Dec 19, 2023

Tested on Android 7 DJI Smart Controller, and Samsung Tab S8 Ultra.

@@ -9,7 +9,7 @@ if (keystorePropertiesFile.exists()) {
}

android {
compileSdk = 34
compileSdk 33
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it required to drop down the SDK version to 33?
Are there any ways to keep SDK 34 and run it successfully on Android 13 devs?

@Peter-St Peter-St merged commit 11da459 into Peter-St:master Dec 19, 2023
@Peter-St
Copy link
Owner

I welcome your decision to commit to Android 13 permissions in the Repo.
I will merge this request to fix this permission issue.

@nsknyc
Copy link
Contributor Author

nsknyc commented Dec 19, 2023

I believe I had an older gradle with 28, and put 33. I can test 34 today no problem. This should technically allow android 14 users as well, but Samsung has not released an official upgrade in the US for the tab s8 for me to test it.

@nsknyc
Copy link
Contributor Author

nsknyc commented Dec 19, 2023

As expected SDK 34 no issues on a 13 device.

@Peter-St
Copy link
Owner

Thx for the conformation.

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.

2 participants