Skip to content

Make build label open Play Store - #3854

Merged
andreia-ferreira merged 3 commits into
google:masterfrom
arpitagarwal1301:fix/3852-build-label-play-store
Jul 24, 2026
Merged

Make build label open Play Store#3854
andreia-ferreira merged 3 commits into
google:masterfrom
arpitagarwal1301:fix/3852-build-label-play-store

Conversation

@arpitagarwal1301

Copy link
Copy Markdown
Contributor

Fixes #3852

  • Make the build label in the navigation drawer a full-row accessible button.
  • Open the app's Play Store listing, with an HTTPS fallback when no market app is available.
  • Add focused coverage for the click target, market intent, and fallback behavior.

Verification

  • ./gradlew checkCode assembleLocalDebug
  • ./gradlew :app:testLocalDebugUnitTest --tests 'org.groundplatform.android.ui.home.HomeScreenFragmentTest'
  • git diff --check

Limitations

  • The external Play Store transition was not exercised on a physical device or emulator; intent behavior is covered with Robolectric.

Comment thread app/src/main/java/org/groundplatform/android/ui/home/HomeScreenFragment.kt Outdated
Comment thread app/src/test/java/org/groundplatform/android/ui/home/HomeScreenFragmentTest.kt Outdated
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 68.42105% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.38%. Comparing base (f5984d1) to head (ac3106d).

Files with missing lines Patch % Lines
...oundplatform/android/ui/home/HomeScreenFragment.kt 73.33% 4 Missing ⚠️
...a/org/groundplatform/android/ui/home/HomeDrawer.kt 50.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3854      +/-   ##
============================================
+ Coverage     67.34%   67.38%   +0.03%     
- Complexity     1836     1839       +3     
============================================
  Files           407      407              
  Lines         10742    10755      +13     
  Branches       1416     1418       +2     
============================================
+ Hits           7234     7247      +13     
+ Misses         2763     2762       -1     
- Partials        745      746       +1     
Files with missing lines Coverage Δ
...a/org/groundplatform/android/ui/home/HomeDrawer.kt 79.05% <50.00%> (-0.40%) ⬇️
...oundplatform/android/ui/home/HomeScreenFragment.kt 71.57% <73.33%> (+3.72%) ⬆️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andreia-ferreira
andreia-ferreira merged commit e875c1a into google:master Jul 24, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Build label clickable, redirecting to Playstore

3 participants