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

Updates: target sdk version to 31 #1006

Conversation

AjeshRPai
Copy link
Contributor

Closes #1003

This PR updates the targetSDKversion to 31 for the Android 12 migration

Test Instructions

[x] If there are new strings that have to be translated, I have added them to the client's strings.xml as a part of the integration PR.

@AjeshRPai AjeshRPai self-assigned this Oct 3, 2022
@AjeshRPai AjeshRPai linked an issue Oct 3, 2022 that may be closed by this pull request
This change wont impact the library as this change only affects the Example app. This change has to be made to make sure that the CI passes
From Android 12 the exported attribute of the activity has set to be true/false. Some of the test dependencies was having an outdated version which contains activities not having the exported attribute. The version of the test dependencies are incremented in order to make it compatible with Android 12.
Copy link
Contributor

@fluiddot fluiddot left a comment

Choose a reason for hiding this comment

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

I've tested the editor both as a standalone app and using wordpress-mobile/WordPress-Android#17153 branch in WP-Android, I haven't encountered any issue so looks like these changes are good to go 🎊 .

@AjeshRPai AjeshRPai marked this pull request as ready for review October 13, 2022 08:15
@fluiddot fluiddot merged commit fdcf6ce into trunk Oct 13, 2022
@fluiddot fluiddot deleted the issue/1003-update-targetsdkversion-to-31-for-android-12-migration branch October 13, 2022 09:48
build.gradle Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update targetSdkVersion to 31 for Android 12 Migration
3 participants