Skip to content

fix(ui): one name per measure across the pilot's screens, and patient links under /patients (#648) - #724

Merged
Taleef7 merged 2 commits into
mainfrom
fix/measure-names
Sep 28, 2026
Merged

Taleef7 merged 2 commits into
mainfrom
fix/measure-names

Conversation

@Taleef7

@Taleef7 Taleef7 commented Sep 28, 2026

Copy link
Copy Markdown
Owner

Closes #648.

I rechecked every surface on the live Maui sandbox first (details on the issue). The Cases cards, case detail and the attributed-list report were already right. This PR fixes the rest.

One label per measure

  • The roster's "Scoped to" line read MIPS 113 · CMS130 · cms130, with the id twice and no name. It now uses labelForId.
  • Every measure picker uses the card's long form, MIPS 113 · CMS130 · Colorectal Cancer Screening:
    • Cases, Orders and Campaigns;
    • the hierarchy;
    • groups (the checkboxes and the table);
    • Admin's waiver filters.
  • The campaign history and a person's history use the same label.
  • A measure with no published identity (the TWH occupational ones) keeps its plain name.
  • Work list gap chips show the short form, MIPS 113 · CMS130 (new shortLabelFor), so a row of three still fits its cell. The full label, status and next action are the chip's tooltip and accessible name, and the visible text is where that name starts (WCAG 2.5.3).

Patient links

  • Subject links come from one helper, subjectPath(id): /patients/<id> on a patient deployment, /employees/<id> otherwise. It is used at 10 link sites.
  • The new /patients/[externalId] route serves the same page. /employees/<id> still opens, so bookmarks and TWH are unchanged.
  • The Maui e2e selectors now look for /patients/; CI builds that stack with NEXT_PUBLIC_SUBJECT_TERM=patient.
  • The profile's API call now encodes the id.

Left as they are

  • The Runs list: engineering only, and its name can be a scope label.
  • Admin's waiver cards and delivery log: TWH occupational measures, which have no identity.

Tests

New or changed:

  • the roster banner, and the Cases, Orders and hierarchy option names;
  • Work list chips: short text, long accessible name, and plain name when there is no identity;
  • the People history label, and the People and Cases links on the patient term;
  • lib/subject-path.test.ts: both terms, plus encoding.

13 mutations, all caught. Full frontend suite (585), lint and build pass locally; the build lists /patients/[externalId].

Taleef added 2 commits September 28, 2026 17:43
…d for the deployment (#648)

The compliance roster's "Scoped to" line printed the id twice ("MIPS 113 · CMS130 · cms130"), and the
measure pickers offered plain names where the cards say "MIPS 113 · CMS130 · Colorectal Cancer
Screening". Every picker (Cases, Orders, Campaigns, the hierarchy, groups, Admin waivers) and the
roster line now use the card's label; a measure with no published identity keeps its name.

Work list gap chips carry the short form ("MIPS 113 · CMS130") so a row of three still fits its cell;
the full label, status and next action are the chip's tooltip and accessible name.

Subject links are built by one helper from the deployment's term: /patients/<id> on a patient
deployment. The /patients route serves the same page, and /employees/<id> still opens, so bookmarks and
TWH are unaffected. The Maui e2e selectors follow the new links.
…'s label, the profile encodes its id, the route is documented
@Taleef7 Taleef7 self-assigned this Sep 28, 2026
@Taleef7
Taleef7 merged commit 0e3994b into main Sep 28, 2026
14 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.

A measure is named five different ways across five screens, and the cases card prints the id twice

1 participant