Skip to content

Add Penicillin ADT property for Haemophilus Analysis#204

Draft
Copilot wants to merge 3 commits into
masterfrom
copilot/add-penicillin-adt-property
Draft

Add Penicillin ADT property for Haemophilus Analysis#204
Copilot wants to merge 3 commits into
masterfrom
copilot/add-penicillin-adt-property

Conversation

Copilot AI commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

Adds a new PenicillinAdt enum field (zone diameter in mm) to the Haemophilus isolate model, UI, and export pipeline.

Model & Enum

  • New PenicillinAdt enum: NotDetermined (0), OneEleven (1–11), GreaterThanTwelve (12) with display descriptions
  • Property added to IsolateBase after BetaLactamase — only affects dbo.Isolates, not Meningo

Frontend

  • Added suffix support to EnumRadioEditorFor so the unit label appears once after the button group, not on each button:
@Html.EnumRadioEditorFor(model => model.PenicillinAdt, "col-sm-10", suffix: "mm")

Renders as: Penicillin ADT [ n.d. | 1 | 2 | ... | 11 | ≥ 12 ] mm

Export

  • HaemophilusSendingLaboratoryExport: new field after BetaLactamase with " mm" suffix on the exported description value

Migration

  • 202604170521000_Isolate_PenicillinAdt: adds PenicillinAdt int column to dbo.Isolates only

Tests

  • 3 new export tests covering normal value ("6 mm"), not determined ("n.d. mm"), and boundary ("≥ 12 mm")
  • Updated column count assertion (76 → 77)

Copilot AI and others added 2 commits April 17, 2026 05:29
…ion (dbo.Isolates only), and unit tests

Agent-Logs-Url: https://github.com/markusrt/NRZMHiDB/sessions/8c0dfc8e-7d42-4db2-9dbe-f9ff4c0bccec

Co-authored-by: markusrt <299753+markusrt@users.noreply.github.com>
Copilot AI changed the title [WIP] Add Penicillin ADT as property for Haemophilus Analysis Add Penicillin ADT property for Haemophilus Analysis Apr 17, 2026
Copilot AI requested a review from markusrt April 17, 2026 05:34
@sonarqubecloud

Copy link
Copy Markdown

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.

Add Penicillin ADT as property for Haemophilus Analysis

2 participants