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.

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.

renamed. thanks!

renamed

updated to not have it update always

updated with latest staging implementation.

Added but crucible is not showing this change. Please check bitbucket.

These variables are correctly initialized to the first enum in the list.

Not needed because the retry mechanism is going to be deleted in the next ticket.

I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.

I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.

Not needed because the retry mechanism is going to be deleted in the next ticket.

What is happening here? Not clear if we need cartridge param. Let's remove it for now.

Remove extra blank line.

Remove extra blank line.

Comment these out again.

Move this to right before syringe pump controller below. Add comment "Execute pinch valve command state machine."

Move this to right after valves controller exec below.

Restore this.