Comment Search

Added.

added

Let all public functions come on top for all files

Added

Fixed.

For testing the patient ID just test 2 different scenarios.
1. Test with a ptient ID (ideally with the full character limit) then enter all parameters then validate

• Can test the validator only able to enter character that are acceptable..
  2. Test with NO patient ID then enter all parameters then validate.

No need to test 10 different patient IDs just those scenarios listed

Published in the respective modules

Need to co-ordinate with drybicart message id since it is using 0x70

The alarm data doesn't sound correct.

I guess my point was, we don't need the "APP_SERVICES" part in the name.

yes. they are added.

Renamed the function to match the staging

where are we calling an alarm?
i think we are only sending event change data in completeBolusToCumulative() right?

Should we request pre-treatment mode here instead? How do we get to pre-treatment mode?

Missing return statement.

Changed. Thank you.

fixed it

removed

Integer 'i' is 4 bytes. Is that what you mean?

Message handler functions must have a single parameter (message). The payload of the message will have the param and value in it and this function needs to parse them out of the payload. Also need to check payload length first.
We need a new message ID in MsgDefs.h for Dialin to send us this message and we need to add an entry in the message/handler table in Messaging.c.

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.

method name is updated has setObjectText()

Done

Feature Configurations to match system.conf

Remove extra spaces between { and msg name.