Skip to content

Sc 17708 #5753

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Gyan-Gupta-Rtsl wants to merge 5 commits into
base: dm-dashboard-new-controlled-bp-statin-cards
Choose a base branch
from
Open

Sc 17708 #5753

Gyan-Gupta-Rtsl wants to merge 5 commits into from

Conversation

@Gyan-Gupta-Rtsl
Copy link
Contributor

@Gyan-Gupta-Rtsl Gyan-Gupta-Rtsl commented Jan 15, 2026 •
edited
Loading

Story card: sc-17708

Because

The DM dashboard "DM patients with controlled BP" card was using dummy data and needed backend support to display real metrics.

This addresses

  1. Adds backend methods in RegionSummarySchema to calculate DM patients with controlled BP (<140/90 and <130/80)
  2. Wires DmBpControlledComponent to use real data from RepositoryPresenter
  3. Adds specs for the new metrics

Test instructions

Suit test cases.

@Gyan-Gupta-Rtsl Gyan-Gupta-Rtsl changed the base branch from master to dm-dashboard-new-controlled-bp-statin-cards January 15, 2026 03:25
@Gyan-Gupta-Rtsl Gyan-Gupta-Rtsl changed the title Sc 17708 Draft: Sc 17708 Jan 15, 2026
@Gyan-Gupta-Rtsl Gyan-Gupta-Rtsl changed the title Draft: Sc 17708 Sc 17708 Jan 16, 2026
@Gyan-Gupta-Rtsl Gyan-Gupta-Rtsl requested a review from a team January 19, 2026 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jamiecarter7 jamiecarter7 Awaiting requested review from jamiecarter7

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Gyan-Gupta-Rtsl