Skip to content

Fixed HR setup bug leading to no automated HR readings - #17

Merged
saksham2001 merged 2 commits into
mainfrom
fix/colmi-hr-enable-and-activity-samples
Jun 24, 2026
Merged

Fixed HR setup bug leading to no automated HR readings#17
saksham2001 merged 2 commits into
mainfrom
fix/colmi-hr-enable-and-activity-samples

Conversation

@saksham2001

Copy link
Copy Markdown
Owner

Summary

While testing the newly added support for the Colmi ring (R11), I found a bug where the app was not sending the command for automated HR recording, and on syncs the ring was only sending SpO2, HRV and temperature.

Related issues

Type of change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 📟 New / improved wearable support (BLE driver layer)
  • 🤖 Coach / LLM change (tools, prompts, orchestration)
  • 🎨 UI / DesignSystem change
  • 🧹 Refactor / chore (no behavior change)
  • 📝 Docs only
  • ⚠️ Breaking change (existing data, settings, or APIs change)

How was this tested?

  • Added / updated unit tests (PulseLoopTests)
  • Ran the test suite locally (⌘U in Xcode)
  • Tested on a physical device with a real ring — model: Colmi R11 (same hardware as R02)
  • Tested with demo data (-seedDemo YES, no hardware)
  • N/A (docs / non-code change)

Privacy & data

  • This change does not send health data off-device without explicit user action.
  • No secrets, API keys, or personal data are committed.
  • N/A

Screenshots / recordings

Checklist

  • My code follows the project's style (SwiftLint passes).
  • I ran the tests and they pass.
  • I updated docs / README where relevant.
  • I read the Contributing guide.

@saksham2001
saksham2001 merged commit 26a6075 into main Jun 24, 2026
2 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.

1 participant