· /sources/gui/qml/components/ModalDialog.qml	(-13, +13)	| History	| Source	| Diff
· /sources/gui/qml/components/UltrafiltrationButton.qml	(-1, +1)	| History	| Source	| Diff
· /sources/gui/qml/compounds/NumPad.qml	(-2, +2)	| History	| Source	| Diff
· /sources/gui/qml/dialogs/AlarmDialog.qml	(-11, +11)	| History	| Source	| Diff
· /sources/gui/qml/dialogs/NotificationDialog.qml	(-1, +1)	| History	| Source	| Diff
· /sources/gui/qml/main.qml	(-1, +1)	| History	| Source	| Diff
· /sources/gui/qml/pages/settings/SettingsAdvancedInstitutional.qml	(-12, +12)	| History	| Source	| Diff
· /sources/gui/qml/pages/settings/SettingsCalibrationSettings.qml	(-12, +12)	| History	| Source	| Diff
· /sources/gui/qml/pages/settings/SettingsInstitutionalRecord.qml	(-12, +12)	| History	| Source	| Diff
· /sources/gui/qml/pages/treatment/TreatmentHDF.qml	(-1, +1)	| History	| Source	| Diff
· /sources/gui/qml/pages/treatment/TreatmentHeparin.qml	(-1, +1)	| History	| Source	| Diff
· /sources/gui/qml/pages/treatment/TreatmentSection.qml	(-13, +13)	| History	| Source	| Diff
· /sources/gui/qml/pages/treatment/TreatmentTrends.qml	(-16, +16)	| History	| Source	| Diff
· /sources/gui/qml/pages/treatment/sections/TreatmentFlows.qml	(-1, +1)	| History	| Source	| Diff
· /sources/gui/qml/pages/treatment/sections/TreatmentFlowsComponent.qml	(-12, +12)	| History	| Source	| Diff
· /sources/gui/qml/pages/treatment/sections/TreatmentHDFComponent.qml	(-1, +1)	| History	| Source	| Diff
· /sources/gui/qml/plugins/virtualkeyboard/layouts/fallback/symbols.qml	(-1, +1)	| History	| Source	| Diff

]]> Wed, 19 Aug 2026 15:04:48 -0700 Dara Navaei https://devapps.diality.us/cru/LEAHI-APPLICATION-LDT-5618-1 https://devapps.diality.us/cru/LEAHI-APPLICATION-LDT-5618-1 Dara Navaei started review LEAHI-APPLICATION-LDT-5618-1 ]]> Wed, 19 Aug 2026 13:51:31 -0700 Nicholas Ramirez https://devapps.diality.us/cru/LEAHI-APPLICATION-LDT-5158-1/#c31512 https://devapps.diality.us/cru/LEAHI-APPLICATION-LDT-5158-1/#c31512 Nicholas Ramirez replied to Behrouz NematiPour in LEAHI-APPLICATION-LDT-5158-1

updated

APPROVED.

RESOLVED.
Added another comment regarding the same.

Nicholas Ramirez
Thanks for the change to the ok and result.

• To be consistent, could you please use ok instead of result, since all the other parts of the code use ok as an argument, and the function itself has ok as the parameter name.
• Also move the:

  ; if ( ! ok ) { goto invalid; }
  

  on the "same line" and "aligned" as the evaluating parameter, to keep the code clean.

APPROVED

updated

updated

updated

remove the ok and just check result and if any fails goto invalid

see if you can use payloadLen from MessageGlobals.h

double check

removedin https://devapps.diality.us/cru/#LEAHI-APPLICATION-LDT-5331-1CFR-87504

remove from prime // priming - TODO check if needed. may not need

BN APPROVES

BN APPROVES

BN APPROVES

BN APPROVES