medic / cht-android

A native Android container for Community Health Toolkit (CHT) applications
GNU Affero General Public License v3.0
25 stars 51 forks source link

Update target API to latest #324

Closed garethbowen closed 1 year ago

garethbowen commented 1 year ago

From Google Play Store

App must target Android 13 (API level 33) or higher To provide users with a safe and secure experience, Google Play requires all apps to meet target API level requirements. From 31 Aug 2023, if your target API level is not within one year of the latest Android release, you won't be able to update your app.

In other words, after August 31 we won't be able to publish updated (or presumably new) android apps until the target API is at least 33.

This has the potential to block new deployments going live so we need to fix this as soon as possible before then.

ngaruko commented 1 year ago

Testing apk from PR : (WIP)

App installed successfully | Android 10 (SDK 29 ) | Android 12 (SDK 31) | Android 13 (SDK 33) | | ------------- | ------------- | ------------- | | image | image | image |
Location Prompt | Android 10 (SDK 29 ) | Android 12 (SDK 31) | Android 13 (SDK 33) | | ------------- | ------------- | ------------- | | image | image | image |
Storage Access Prompt | Android 10 (SDK 29 ) | Android 12 (SDK 31) | Android 13 (SDK 33) - NO PROMPT| | ------------- | ------------- | ------------- | | image| image | image |
Files uploaded successfully | Android 10 (SDK 29 ) | Android 12 (SDK 31) | Android 13 (SDK 33) | | ------------- | ------------- | ------------- | | image | image|image |
latin-panda commented 1 year ago

@ngaruko I found that to test sending reports by SMS, the permission needs to be enabled in the manifest, by default is commented.

Screenshot of SMS that was sent after submitting report
latin-panda commented 1 year ago

@ngaruko I did these same cases you tested in Android 9 and 13, with these additional cases:

ngaruko commented 1 year ago

@latin-panda I have noticed the buck button not closing the app. Did the app launcher with the old MRDT app on Android 13 and 12. All good.

garethbowen commented 1 year ago

@latin-panda Can we close this now?