We can set bolus volume here now. These functions should be moved to TxParams where all of these settings exist and can be updated.

Why does confirm msg have duration again? If UI resends duration, we have to validate it again. If UI just sends confirmation of previously sent duration, we don't have to validate it again.

These are already in TxParams I believe.

Should these be moved to Services\TxParams?

Change 1 to TRUE

Agree with name change, but I think BOOL was the correct type. And BOOL is equivalent to U32 anyway so compiler won't care.

I think these message handlers for editing treatment parameters should be moved to Services\TxParams.

These should probably be moved to Services\TxParams.h because Messaging.c does not use them.

See if we have already defined this stuff in Services\TxParams. TxParams may also have get function(s) for min/max values that you can use.

Has integrity unit been added to TD project?

Where are these being triggered?

Is this necessary? If so, why?

Keep the first condition 'isTestingActivated' check for allowing state changes in testing mode

Fixed

Please update the comments for Pressure alarm and Diener pump.

Fixed.

Fixed.

Fixed.

When Systems gives me Leahi terms, they will be very small fractions.

Fixed.

You're right. Old comment from Denali. Fixed.

Came from Denali, so I believe this is what they wanted. The nearly 5 minutes is the one the user sees.