Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: label jring heart-rate capability as 'HR' to match other rings - #63

Merged
saksham2001 merged 1 commit into
saksham2001:mainfrom
rgvxsthi:fix/jring-hr-label
Jul 8, 2026
Merged

saksham2001 merged 1 commit into
saksham2001:mainfrom
rgvxsthi:fix/jring-hr-label

Conversation

@rgvxsthi

@rgvxsthi rgvxsthi commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Problem

On the ring-pairing carousel, the jring model's capability chips showed Heart rate, while every other ring in the catalog uses the abbreviated HR. Inconsistent labeling for a single model.

Fix

One-line change in WearableModel.swift: jring blurb Heart rate · SpO₂ · SleepHR · SpO₂ · Sleep.

Scope

Display-only string change (capability blurb, split into chips). No API, schema, or behavior change. Independent of the Liquid Glass work (PR #62).

The jring model's capability blurb read 'Heart rate · SpO₂ · Sleep' while
every other ring in the catalog uses the abbreviated 'HR'. This showed an
inconsistent 'Heart rate' chip on the pairing carousel for jring only.
Align it to 'HR'.
@rgvxsthi
rgvxsthi requested a review from saksham2001 as a code owner July 8, 2026 11:40
@saksham2001
saksham2001 merged commit 748e79f into saksham2001:main Jul 8, 2026
2 checks passed
@rgvxsthi
rgvxsthi deleted the fix/jring-hr-label branch July 16, 2026 04:21
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.

2 participants