Skip to content
This repository was archived by the owner on Jul 10, 2025. It is now read-only.

Conversation

@AjeshRPai
Copy link
Contributor

@AjeshRPai AjeshRPai commented Sep 29, 2022

Closes #723

This PR updates the target SDK version to 31 for the Android 12 migration

Test Instructions

  • Install the app from the Parent PR
  • Make sure the functionalities provided by the library work as expected

@AjeshRPai AjeshRPai added the [Type] Enhancement An improvement to existing functionality. label Sep 29, 2022
@AjeshRPai AjeshRPai self-assigned this Sep 29, 2022
@AjeshRPai AjeshRPai linked an issue Sep 29, 2022 that may be closed by this pull request
This change wouldn't affect the libary as this is only on the example app. Change was necessary because of the CI failure
@peril-automattic
Copy link

You can test the changes on this Pull Request by downloading the APK here.

@AjeshRPai AjeshRPai requested a review from ParaskP7 October 13, 2022 08:36
@AjeshRPai AjeshRPai marked this pull request as ready for review October 13, 2022 08:36
Copy link
Contributor

@ParaskP7 ParaskP7 left a comment

Choose a reason for hiding this comment

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

LGTM! 💯

As per the testing that was done as part of WordPress-Android#17153, which verifies correctness of this library working alongside the WPAndroid client app, I am approving this PR too.

PS: I also quickly smoke tested the example loop app.

@AjeshRPai AjeshRPai merged commit 4926f0d into trunk Oct 13, 2022
@AjeshRPai AjeshRPai deleted the issue/723-update-targetsdkversion-and-compilesdkversion-to-31 branch October 13, 2022 10:11
@ParaskP7 ParaskP7 self-assigned this Oct 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

[Type] Enhancement An improvement to existing functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update targetSdkVersion and compileSdkVersion to 31

2 participants