Updated

Fixed.

Moved.

Move buttons down just above "Comm.h". And why is buttons being included here?

This reason code is not used anywhere, and the existing REQUEST_REJECT_REASON_NOT_ALLOWED_IN_CURRENT_MODE can be used if a rejection response is needed. I removed the new reason code.

I don't see it.

I don't see it.

I would think reason code 1 would work for this - do we really need to add this one?

Fix this.

break this line

add an empty else block after the last else if block in these functions
handlePreTxHeparinPreloadState
handlePreTxHeparinAwaitSyringeLoadConfirmationState
handlePreTxHeparinSeekState
handlePreTxHeparinPrimeState
handlePreTxHeparinBolusState

break this line

Updated as suggested by removing the unnecessary else block and reject command. Retested the power-off confirmation and shutdown flow successfully.

Updated to remove the return from the middle of the function. The function now sets newID and returns only once at the end. Retested successfully.

Add initialization for heparinSetupResumeRequested and syringeLoadConfirmed

Align all the function comments

Delete this row.

should we replace current Pre-Treatment state with currentPreTreatmentState?

Add @Message

break this line

break this line

the function call seems returning U16. Type cast to U32.