dimagi · pm-dimagi · Dec 9, 2024 · Dec 9, 2024 · Dec 11, 2024 · Dec 19, 2024
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
@@ -1,29 +1,41 @@
-## Summary
+## Product Description
+<!--
+Delete this section if the PR does not contain any visible changes.
+For non-invisible changes, describe the user-facing effects.
+-->
+
+## Technical Summary
 <!--
-    Provide a link to the ticket or document which prompted this change,
-    Describe the rationale and design decisions.
+- Provide a link to the ticket or document which prompted this change.
+- Describe the rationale and design decisions.
 -->
 
 ## Feature Flag
 <!-- If this is specific to a feature flag, which one? -->
 
-## Product Description
-<!-- For non-invisible changes, describe user-facing effects. Would be good to add screenshots/videos for any major user facing changes -->
+## Safety Assurance
 
-## PR Checklist
+### Safety story
+<!--
+Describe:
+- how you became confident in this change (such as local testing).
+- why the change is inherently safe, and/or plans to limit the defect blast radius.
 
-- [ ] If I think the PR is high risk, "High Risk" label is set
-- [ ] I have confidence that this PR will not introduce a regression for the reasons below
-- [ ] Do we need to enhance manual QA test coverage ? If yes, "QA Note" label is set correctly
-- [ ] Does the PR introduce any major changes worth communicating ? If yes, "Release Note" label is set and a "Release Note" is specified in PR description.
+In particular consider how existing data may be impacted by this change.
+-->
 
 ### Automated test coverage
+<!-- Identify the related test coverage and the conditions it will catch -->
 
-<!-- Identify the related test coverage and the tests it would catch -->
+### QA Plan
+<!--
+- Describe QA plan that (along with test coverage) proves that this PR is regression free.
+- Link to QA Ticket
+-->
 
+## Labels and Review
 
-### Safety story
-<!--
-Describe any other pieces to the safety story including
-local testing, why the change is inherently safe, and/or plans to limit the blast radius of a defect.
---> 
+- [ ] Do we need to enhance the manual QA test coverage ? If yes, the "QA Note" label is set correctly
+- [ ] Does the PR introduce any major changes worth communicating ? If yes, the "Release Note" label is set and a "Release Note" is specified in PR description.
+- [ ] Risk label is set correctly
+- [ ] The set of people pinged as reviewers is appropriate for the level of risk of the change
diff --git a/app/AndroidManifest.xml b/app/AndroidManifest.xml
@@ -27,8 +27,6 @@
     <uses-permission
         android:name="android.permission.READ_EXTERNAL_STORAGE"
         android:maxSdkVersion="32"/>
-    <uses-permission android:name="android.permission.READ_MEDIA_IMAGES" />
-    <uses-permission android:name="android.permission.READ_MEDIA_VIDEO" />
     <uses-permission android:name="android.permission.READ_MEDIA_AUDIO" />
     <uses-permission android:name="android.permission.WAKE_LOCK"/>
     <uses-permission android:name="android.permission.RECORD_AUDIO" />
@@ -175,7 +173,15 @@
         <activity android:name="org.commcare.activities.StandardHomeActivity"/>
         <activity android:name="org.commcare.activities.RootMenuHomeActivity"/>
         <activity android:name="org.commcare.activities.MenuActivity"/>
-
+        <service android:name="com.google.android.gms.metadata.ModuleDependencies"
+            android:enabled="false"
+            android:exported="false"
+            tools:ignore="MissingClass">
+            <intent-filter>
+                <action android:name="com.google.android.gms.metadata.MODULE_DEPENDENCIES" />
+            </intent-filter>
+            <meta-data android:name="photopicker_activity:0:required" android:value="" />
+        </service>
         <provider
             android:name="org.commcare.provider.FormsProvider"
             android:exported="false"

diff --git a/app/assets/locales/android_translatable_strings.txt b/app/assets/locales/android_translatable_strings.txt
@@ -529,6 +529,10 @@ login.attempt.fail.changed.title=Password Changed since Last Login
 login.attempt.fail.changed.detail=You've logged in with a new password since you're last login. You may need to log in with your old credentials to ensure you have no unsent data.
 login.attempt.fail.changed.action=If you had unsent data or incomplete forms on the phone before you logged in with your new password you should log out, and log in with your old password. Make sure all your unsent forms are sent and incomplete forms are submitted, then log in again with your new password and sync. 
 
+login.attempt.insufficient.role.permission.title=User doesn't have permission to access the CommCare application
+login.attempt.insufficient.role.permission.detail=The role assigned to your user doesn't have permission to access CommCare mobile application
+login.attempt.insufficient.role.permission.action=Contact Support for assistance with this matter
+
 notification.credential.mismatch.title=Mismatching Login Credentials
 notification.credential.mismatch.detail=The server recognized your password, but the password it sent to the device does not match 
 notification.credential.mismatch.action=Contact CommCare Support for assistance with your account
@@ -992,3 +996,5 @@ background.sync.fail=Background sync failed. Please try to trigger a normal Sync
 android.package.name.org.commcare.dalvik.reminders=CommCare Reminders
 android.package.name.callout.commcare.org.sendussd=Commcare USSD
 android.package.name.org.commcare.dalvik.abha=CommCare ABHA
+
+location.capture.cancelled=Location capture cancelled
diff --git a/app/build.gradle b/app/build.gradle
@@ -22,6 +22,7 @@ repositories {
         dirs 'libs'
     }
     maven { url 'https://jitpack.io' }
+    maven {url 'https://repository.liferay.com/nexus/content/repositories/public/'}
 }
 
 configurations {
@@ -43,6 +44,8 @@ dependencies {
     testImplementation 'org.jetbrains.kotlinx:kotlinx-coroutines-test:1.7.3'
     testImplementation 'io.mockk:mockk:1.12.7'
     testImplementation 'org.json:json:20231013'
+    testImplementation 'org.mockito:mockito-core:5.5.0'
+    testImplementation 'org.json:json:20140107'
-    testImplementation 'org.mockito:mockito-core:5.5.0'
-    testImplementation 'org.json:json:20140107'
+    testImplementation 'org.json:json:20231013'
+    testImplementation 'org.mockito:mockito-core:5.5.0'
-    testImplementation 'org.mockito:mockito-core:5.5.0'
-    testImplementation 'org.json:json:20140107'
+    testImplementation 'org.json:json:20231013'
+    testImplementation 'org.mockito:mockito-core:5.5.0'
     testImplementation project(path: ':commcare-core', configuration: 'testsAsJar')
 
     androidTestImplementation 'androidx.test:runner:1.4.0'
@@ -68,11 +71,7 @@ dependencies {
     implementation fileTree(include: '*.jar', exclude: 'regexp-me.jar', dir: 'libs')
     implementation(name: 'htmlspanner-custom', ext: 'aar')
     implementation 'com.github.dimagi:zebra-print-android:v1.3'
-    implementation (name: 'LibSimprints-1.0.12', ext: 'aar')
-    implementation (name: 'mapbox-android-accounts-0.2.0', ext: 'aar')
-    implementation (name: 'realm-android-library-4.1.1', ext: 'aar')
-    implementation (name: 'volley-1.1.0', ext: 'aar')
-    implementation (name: 'storage-2.1.0', ext: 'aar')
+    implementation 'com.simprints:libsimprints:2024.2.2'
     implementation 'androidx.appcompat:appcompat:1.2.0'
     implementation 'androidx.cardview:cardview:1.0.0'
     implementation 'androidx.recyclerview:recyclerview:1.1.0'
@@ -84,6 +83,9 @@ dependencies {
     implementation 'joda-time:joda-time:2.9.4'
     implementation 'net.zetetic:android-database-sqlcipher:4.5.3@aar'
     implementation 'androidx.sqlite:sqlite:2.2.0'
+    implementation 'com.squareup.retrofit2:retrofit:2.9.0'
+    implementation 'com.squareup.okhttp3:logging-interceptor:4.11.0'
+    implementation 'com.squareup.retrofit2:converter-gson:2.9.0'
     implementation('org.apache.james:apache-mime4j:0.7.2') {
         exclude module: 'commons-io'
     }
@@ -114,12 +116,13 @@ dependencies {
     implementation "io.noties.markwon:ext-tables:$markwon_version"
     implementation "io.noties.markwon:ext-strikethrough:$markwon_version"
 
+    // Kujaku
+    implementation 'com.android.volley:volley:1.2.1'
+    implementation 'com.snatik:storage:2.1.0'
     implementation('io.ona.kujaku:library:0.9.0') {
         exclude module: 'xpp3'
         exclude module: 'volley'
         exclude module: 'storage'
-        exclude group: 'com.mapbox.mapboxsdk', module: 'mapbox-android-accounts'
-        exclude group: 'io.realm'
     }
     implementation 'androidx.work:work-runtime:2.10.0'
     implementation 'androidx.work:work-runtime-ktx:2.10.0'
@@ -282,9 +285,7 @@ android {
         buildConfigField 'String', 'ANALYTICS_TRACKING_ID_LIVE', "\"${project.ext.ANALYTICS_TRACKING_ID_LIVE}\""
         buildConfigField 'String', 'ANALYTICS_TRACKING_ID_DEV', "\"${project.ext.ANALYTICS_TRACKING_ID_DEV}\""
         buildConfigField 'String', 'MAPBOX_SDK_API_KEY', "\"${project.ext.MAPBOX_SDK_API_KEY}\""
-
         buildConfigField "String", "FIREBASE_DATABASE_URL", "\"${project.ext.FIREBASE_DATABASE_URL}\""
-
         buildConfigField 'String', 'CCC_HOST', "\"connect.dimagi.com\""
 
         testInstrumentationRunner 'org.commcare.CommCareJUnitRunner'

diff --git a/app/instrumentation-tests/src/org/commcare/androidTests/BaseTest.kt b/app/instrumentation-tests/src/org/commcare/androidTests/BaseTest.kt
@@ -48,9 +48,7 @@ abstract class BaseTest {
 
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
             appPermissions.addAll(arrayOf(
-                    Manifest.permission.READ_MEDIA_IMAGES,
                     Manifest.permission.READ_MEDIA_AUDIO,
-                    Manifest.permission.READ_MEDIA_VIDEO,
                     Manifest.permission.POST_NOTIFICATIONS))
         } else {
             appPermissions.addAll(arrayOf(

diff --git a/app/libs/LibSimprints-1.0.12.aar b/app/libs/LibSimprints-1.0.12.aar
diff --git a/app/libs/mapbox-android-accounts-0.2.0.aar b/app/libs/mapbox-android-accounts-0.2.0.aar
diff --git a/app/libs/realm-android-library-4.1.1.aar b/app/libs/realm-android-library-4.1.1.aar
diff --git a/app/libs/storage-2.1.0.aar b/app/libs/storage-2.1.0.aar
diff --git a/app/libs/volley-1.1.0.aar b/app/libs/volley-1.1.0.aar
diff --git a/app/res/drawable-ldpi/ic_blue_backward.png b/app/res/drawable-ldpi/ic_blue_backward.png
diff --git a/app/res/drawable-ldpi/ic_blue_forward.png b/app/res/drawable-ldpi/ic_blue_forward.png
diff --git a/app/res/drawable-ldpi/icon_chevron_left_brand.png b/app/res/drawable-ldpi/icon_chevron_left_brand.png
diff --git a/app/res/drawable-ldpi/icon_chevron_right_brand.png b/app/res/drawable-ldpi/icon_chevron_right_brand.png
diff --git a/app/res/drawable-mdpi/ic_blue_backward.png b/app/res/drawable-mdpi/ic_blue_backward.png
diff --git a/app/res/drawable-mdpi/ic_blue_forward.png b/app/res/drawable-mdpi/ic_blue_forward.png
diff --git a/app/res/drawable-mdpi/icon_chevron_left_brand.png b/app/res/drawable-mdpi/icon_chevron_left_brand.png
diff --git a/app/res/drawable-mdpi/icon_chevron_right_brand.png b/app/res/drawable-mdpi/icon_chevron_right_brand.png
diff --git a/app/res/drawable-xhdpi/ic_blue_backward.png b/app/res/drawable-xhdpi/ic_blue_backward.png
diff --git a/app/res/drawable-xhdpi/ic_blue_forward.png b/app/res/drawable-xhdpi/ic_blue_forward.png
diff --git a/app/res/drawable-xhdpi/icon_chevron_left_brand.png b/app/res/drawable-xhdpi/icon_chevron_left_brand.png
diff --git a/app/res/drawable-xhdpi/icon_chevron_right_brand.png b/app/res/drawable-xhdpi/icon_chevron_right_brand.png
diff --git a/app/res/drawable-xxhdpi/ic_blue_backward.png b/app/res/drawable-xxhdpi/ic_blue_backward.png
diff --git a/app/res/drawable-xxhdpi/ic_blue_forward.png b/app/res/drawable-xxhdpi/ic_blue_forward.png
diff --git a/app/res/drawable/icon_auto_advance_arrow.png b/app/res/drawable/icon_auto_advance_arrow.png
diff --git a/app/res/drawable/icon_chevron_left_brand.png b/app/res/drawable/icon_chevron_left_brand.png
diff --git a/app/res/drawable/icon_chevron_left_primary.png b/app/res/drawable/icon_chevron_left_primary.png
diff --git a/app/res/drawable/icon_chevron_right_brand.png b/app/res/drawable/icon_chevron_right_brand.png
diff --git a/app/res/drawable/icon_chevron_right_primary.png b/app/res/drawable/icon_chevron_right_primary.png
diff --git a/app/res/drawable/icon_new.png b/app/res/drawable/icon_new.png
diff --git a/app/res/drawable/icon_next.png b/app/res/drawable/icon_next.png
diff --git a/app/res/layout/entity_select_layout.xml b/app/res/layout/entity_select_layout.xml
@@ -53,6 +53,7 @@
         android:visibility="gone"/>
 
     <RelativeLayout
+        android:id="@+id/progress_container"
         android:layout_width="match_parent"
         android:layout_height="fill_parent"
         android:layout_below="@id/entity_select_filter_dropdown">
@@ -63,8 +64,18 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:layout_centerInParent="true"/>
+
+        <TextView
+            android:id="@+id/progress_text"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_below="@+id/entity_select_loading"
+            android:padding="8dp"
+            android:layout_centerInParent="true" />
+
     </RelativeLayout>
 
+
     <FrameLayout
         android:layout_width="match_parent"
         android:layout_height="wrap_content"

diff --git a/app/res/layout/screen_form_entry.xml b/app/res/layout/screen_form_entry.xml
@@ -70,7 +70,6 @@
             android:id="@+id/nav_btn_prev"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:layout_centerVertical="true"
             android:layout_alignParentLeft="true"
             android:background="@drawable/selector_button_press"
             android:minHeight="@dimen/new_progressbar_button_minheight"
@@ -87,8 +86,7 @@
             android:id="@+id/nav_prog_bar"
             android:indeterminate="false"
             android:indeterminateOnly="false"
-            android:layout_centerVertical="true"
-            android:layout_height="32dp"
+            android:layout_height="wrap_content"
             android:layout_marginEnd="@dimen/content_start"
             android:layout_marginLeft="@dimen/content_start"
             android:layout_marginRight="@dimen/content_start"
@@ -99,8 +97,8 @@
             android:layout_toStartOf="@+id/nav_btn_next"
             android:layout_width="wrap_content"
             android:max="100"
-            android:paddingBottom="@dimen/content_start"
-            android:paddingTop="@dimen/content_start"
+            android:paddingBottom="@dimen/progressbar_vertical_padding"
+            android:paddingTop="@dimen/progressbar_vertical_padding"
             android:progress="0"
             android:progressDrawable="@drawable/progressbar_modern" />
 
@@ -110,7 +108,6 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:layout_alignParentRight="true"
-            android:layout_centerVertical="true"
             android:background="@drawable/selector_button_press"
             android:minHeight="@dimen/new_progressbar_button_minheight"
             android:minWidth="@dimen/new_progressbar_button_minwidth"

diff --git a/app/res/layout/square_button_text.xml b/app/res/layout/square_button_text.xml
@@ -26,8 +26,8 @@
         android:paddingBottom="@dimen/standard_spacer"
         android:paddingLeft="@dimen/standard_spacer"
         android:paddingRight="@dimen/standard_spacer"
-        android:textSize="@dimen/font_size_xlarge"
-        app:text_size_small="@dimen/font_size_large"
+        android:textSize="@dimen/font_size_large"
+        app:text_size_small="@dimen/font_size_medium"
         android:paddingStart="@dimen/standard_spacer"
         android:paddingEnd="@dimen/standard_spacer" />
 </RelativeLayout>
diff --git a/app/res/layout/square_card.xml b/app/res/layout/square_card.xml
@@ -41,8 +41,8 @@
                 android:paddingBottom="@dimen/standard_spacer"
                 android:paddingLeft="@dimen/standard_spacer"
                 android:paddingRight="@dimen/standard_spacer"
-                android:textSize="@dimen/font_size_xlarge"
-                app:text_size_small="@dimen/font_size_large"
+                android:textSize="@dimen/font_size_large"
+                app:text_size_small="@dimen/font_size_medium"
                 android:paddingStart="@dimen/standard_spacer"
                 android:paddingEnd="@dimen/standard_spacer" />
         </RelativeLayout>
@@ -54,6 +54,6 @@
             android:layout_below="@+id/card"
             android:layout_gravity="end"
             android:padding="@dimen/help_text_padding"
-            android:textSize="@dimen/font_size_medium"/>
+            android:textSize="@dimen/font_size_small"/>
     </RelativeLayout>
 </androidx.cardview.widget.CardView>
diff --git a/app/res/values-w320dp/dimens.xml b/app/res/values-w320dp/dimens.xml
@@ -1,10 +1,10 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
     <dimen name="edit_text_question_widget_compact_width">100dp</dimen>
-    <dimen name="font_size_small">12sp</dimen>
-    <dimen name="font_size_medium">15sp</dimen>
-    <dimen name="font_size_large">18sp</dimen>
-    <dimen name="font_size_xlarge">21sp</dimen>
+    <dimen name="font_size_small">14sp</dimen>
+    <dimen name="font_size_medium">17sp</dimen>
+    <dimen name="font_size_large">19sp</dimen>
+    <dimen name="font_size_xlarge">23sp</dimen>
     <dimen name="cell_padding_vertical">8sp</dimen>
     <dimen name="cell_padding_horizontal">8sp</dimen>
     <dimen name="row_padding_vertical">8sp</dimen>

diff --git a/app/res/values-w600dp/dimens.xml b/app/res/values-w600dp/dimens.xml
@@ -2,9 +2,9 @@
 <resources>
     <dimen name="edit_text_question_widget_compact_width">200dp</dimen>
     <dimen name="font_size_small">16sp</dimen>
-    <dimen name="font_size_medium">20sp</dimen>
-    <dimen name="font_size_large">24sp</dimen>
-    <dimen name="font_size_xlarge">28sp</dimen>
+    <dimen name="font_size_medium">18sp</dimen>
+    <dimen name="font_size_large">25sp</dimen>
+    <dimen name="font_size_xlarge">30sp</dimen>
     <dimen name="cell_padding_vertical">8sp</dimen>
     <dimen name="cell_padding_horizontal">8sp</dimen>
     <dimen name="row_padding_vertical">8sp</dimen>

diff --git a/app/res/values-w720dp/dimens.xml b/app/res/values-w720dp/dimens.xml
@@ -2,9 +2,9 @@
 <resources>
     <dimen name="edit_text_question_widget_compact_width">240dp</dimen>
     <dimen name="login_box_margins">120dp</dimen>
-    <dimen name="font_size_small">20sp</dimen>
-    <dimen name="font_size_medium">24sp</dimen>
-    <dimen name="font_size_large">28sp</dimen>
+    <dimen name="font_size_small">18sp</dimen>
+    <dimen name="font_size_medium">19sp</dimen>
+    <dimen name="font_size_large">26sp</dimen>
     <dimen name="font_size_xlarge">32sp</dimen>
     <dimen name="cell_padding_vertical">8sp</dimen>
     <dimen name="cell_padding_horizontal">8sp</dimen>

diff --git a/app/res/values/dimens.xml b/app/res/values/dimens.xml
@@ -57,10 +57,10 @@
 
     <dimen name="activity_horizontal_margin">16dp</dimen>
     <dimen name="activity_vertical_margin">16dp</dimen>
-    <dimen name="font_size_small">8sp</dimen>
-    <dimen name="font_size_medium">10sp</dimen>
-    <dimen name="font_size_large">12sp</dimen>
-    <dimen name="font_size_xlarge">14sp</dimen>
+    <dimen name="font_size_small">10sp</dimen>
+    <dimen name="font_size_medium">14sp</dimen>
+    <dimen name="font_size_large">16sp</dimen>
+    <dimen name="font_size_xlarge">21sp</dimen>
     <dimen name="font_size_xlarger">23sp</dimen>
     <dimen name="font_size_xlargest">28sp</dimen>
     <dimen name="font_size_40">40sp</dimen>
@@ -108,6 +108,7 @@
     <dimen name="new_progressbar_button_minwidth">32dp</dimen>
     <dimen name="new_progressbar_button_padding">24dp</dimen>
     <dimen name="modern_progressbar_radius">4dp</dimen>
+    <dimen name="progressbar_vertical_padding">16dp</dimen>
 
     <dimen name="submission_progressbar_height">10dp</dimen>
 

diff --git a/app/res/values/strings.xml b/app/res/values/strings.xml
@@ -11,7 +11,9 @@
     <string name="key_server">https://pact.dimagi.com/keys/getkey</string>
     <string name="your_comment">Your comment</string>
     <string name="support_email_address_default">commcarehq-support@dimagi.com</string>
-
+    <string name="ConnectFetchDbKeyURL">/users/fetch_db_key</string>
+    <string name="ConnectTokenURL">/o/token/</string>
+    <string name="ConnectHeartbeatURL">/users/heartbeat</string>
     <!-- region: All strings for multiple apps and app-agnostic properties -->
 
     <string name="manager_activity_name">App Manager</string>
@@ -389,6 +391,7 @@
     <string name="recovery_forms_send_error" cc:translatable="true">Error while sending forms</string>
     <string name="recovery_forms_state_unavailable" cc:translatable="true">Forms are not available. Make sure your phone storage is available</string>
     <string name="recovery_network_unavailable" cc:translatable="true">No network connection. Please check your internet and try again.</string>
+    <string name="recovery_network_outdated" cc:translatable="true">The app is outdated and can no longer communicate with the server. Please update the app on the Google Play Store.</string>
     <string name="recovery_app_manager" cc:translatable="true">Go to App Manager</string>
     <string name="recovery_retry" cc:translatable="true">Retry Recovery</string>
 
@@ -461,4 +464,14 @@
     <string name="fcm_default_notification_channel">notification-channel-push-notifications</string>
     <string name="app_with_id_not_found">Required CommCare App is not installed on device</string>
     <string name="audio_recording_notification">Audio Recording Notification</string>
+
+    <!--    Entity List Progresss Indicators-->
+    <string name="entity_list_initializing" cc:translatable="true">Initializing list…</string>
+    <string name="entity_list_processing" cc:translatable="true">Processing cases: %s</string>
+    <string name="entity_list_loading_cache" cc:translatable="true">Loading Cache…</string>
+    <string name="entity_list_calculating" cc:translatable="true">Calculating cases %s</string>
+    <string name="entity_list_finishing" cc:translatable="true">Finishing case calculations…</string>
+
+
+    <string name="connect_db_corrupt" cc:translatable="true">A problem occurred with the database, please recover your account.</string>
 </resources>