Skip to content

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
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