Week 10
Mostly review follow-up this week, but three new PRs went up, all surfaced while going back through older work rather than from new tickets, plus a design out for community feedback.
-
content-referenceapplication-demo#79 (merged) — Wires the three privileges from queue#118 onto the demo’s existing roles (Doctor/Nurse → Clinician, Registration Clerk → Clerk, System Administrator → Clinic Administrator), so the Week 7 privilege-gated view has something real to check against in a deployment.
-
queue#120 (merged) — Fixes O3-5760:
QueueUtils.computeAverageWaitTimeInMinutesguarded against an empty entry list but not against a non-empty list where nothing had actually finished waiting yet, leaving the denominator at zero and the average wait time metric renderingNaN. Now returnsnullinstead, which the frontend already handles gracefully. -
patient-chart#3509 (open) — Fixes O3-5763:
VisitNotesFormpulled its visit context fromusePatientChartStore, which only the patient chart populates. Launched from Service Queues,visitContextcame backnull, and the backend’s fallback visit-assignment logic picked a visit, not necessarily the one being viewed. Splits the form the way the vitals form already handles this, takingvisitContextexplicitly as a prop and exporting it as a reusable workspace.
Also put together a design for a dedicated Clinic Administrator screen and posted it for community feedback — a clinic-wide monitoring view, addressing O3-5770, that shows every queue at a glance instead of one at a time. It’s the natural next step after Week 7’s privilege model.
Full write-up on Medium: The Week of “While I Was In There”
Where Things Stand (Updated)
Merged: #2540, #2541, #2543, content-referenceapplication-demo#74, #2559, content-referenceapplication-demo#79, queue#120
Open for review: #2550, #2555, #2564, queue#115, queue#116, #2583, queue#117, patient-chart#3419, #2600, #2608, #2611, #2615, #2625, queue#118, #2626, queue#119, #2633, patient-chart#3509
Draft: #2632 (blocked on queue#119)
Superseded/closed: queue#113, queue#114, #2607
Design out for feedback: Clinic Administrator screen (O3-5770)
Will keep this thread updated as the weeks go on ![]()