Skip to content

[WearOS] Count bluetooth connection as online state#5086

Merged
geekygecko merged 2 commits intorelease/8.7from
fix/wearos-offline-indicator
Mar 5, 2026
Merged

[WearOS] Count bluetooth connection as online state#5086
geekygecko merged 2 commits intorelease/8.7from
fix/wearos-offline-indicator

Conversation

@sztomek
Copy link
Contributor

@sztomek sztomek commented Mar 5, 2026

Description

As title says.
Internal discussion: pdeCcb-c0h-p2#comment-8998

Fixes PCDROID-480

Testing Instructions

  1. Install wearOS app and sign in
  2. while still connected to the phone, disable wifi and cellular connection of the watch
  3. open app
  4. notice the offline indicator is not displayed
  5. now disable bluetooth on your phone
  6. go back to the app
  7. notice offline indicator is visible

Checklist

  • If this is a user-facing change, I have added an entry in CHANGELOG.md
  • Ensure the linter passes (./gradlew spotlessApply to automatically apply formatting/linting)
  • I have considered whether it makes sense to add tests for my changes
  • All strings that need to be localized are in modules/services/localization/src/main/res/values/strings.xml
  • Any jetpack compose components I added or changed are covered by compose previews
  • I have updated (or requested that someone edit) the spreadsheet to reflect any new or changed analytics.

@sztomek sztomek added this to the 8.7 ❄️ milestone Mar 5, 2026
@sztomek sztomek requested a review from a team as a code owner March 5, 2026 15:27
@sztomek sztomek requested review from MiSikora and Copilot and removed request for a team March 5, 2026 15:27
@sztomek sztomek added [Type] Bug Not functioning as intended. [Area] Wear OS watch app labels Mar 5, 2026
@sztomek sztomek requested a review from geekygecko March 5, 2026 15:27
@dangermattic
Copy link
Collaborator

dangermattic commented Mar 5, 2026

1 Warning
⚠️ This PR is assigned to the milestone 8.7 ❄️. The due date for this milestone has already passed.
Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by 🚫 Danger

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR makes WearOS treat Bluetooth connections as a valid online state, so that when a watch is connected to a phone via Bluetooth (but has no WiFi or cellular), the app no longer shows the offline indicator. Previously, only WiFi and Cellular were considered valid connection types.

Changes:

  • Added NetworkType.BT as a valid connected network type in ConnectivityStateManager
  • Updated and added tests to verify Bluetooth is treated as connected and transitions work correctly
  • Bumped version to 8.7-rc-1 / 9411 and updated release notes and CHANGELOG for the 8.7 release cycle

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
wear/src/main/kotlin/.../ConnectivityStateManager.kt Added NetworkType.BT to the connectivity check condition
wear/src/test/kotlin/.../ConnectivityStateManagerTest.kt Updated existing BT test to expect true, added new test for BT disconnect transition
version.properties Bumped version name and code
CHANGELOG.md Added empty 8.8 section header
metadata/release_notes.txt Updated release notes content for 8.7

You can also share your feedback on Copilot code review. Take the survey.

Comment on lines +1 to +4
8.8
-----


Copy link

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

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

The CHANGELOG.md checklist item in the PR description is checked ("I have added an entry in CHANGELOG.md"), but the 8.8 section is empty. There should be an entry describing this change, e.g., under Bug Fixes or Updates, noting that Bluetooth connections are now counted as online state on WearOS.

Copilot uses AI. Check for mistakes.
@geekygecko geekygecko changed the base branch from main to release/8.7 March 5, 2026 23:55
@geekygecko
Copy link
Member

I believe this PR was meant to target the release branch instead of main so I have changed it.

@geekygecko geekygecko merged commit d8e89c6 into release/8.7 Mar 5, 2026
26 of 28 checks passed
@geekygecko geekygecko deleted the fix/wearos-offline-indicator branch March 5, 2026 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Area] Wear OS watch app [Type] Bug Not functioning as intended.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants