· /AlarmDefs.h	(-2, +2)	| History	| Source	| Diff
· /MsgDefs.h	(-2, +2)	| History	| Source	| Diff
· /TDDefs.h	(-2, +2)	| History	| Source	| Diff

]]> Fri, 31 Jul 2026 10:56:39 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4040-1 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4040-1 Sean Nash resumed reviewing LEAHI-TD-FIRMWARE-LDT-4040-1 LEAHI-TD-FIRMWARE-LDT-4040_Rinseback - FW Implementation - 3/10: ]]> Fri, 31 Jul 2026 10:56:17 -0700 Vendor - TEL - Praneeth Bunne https://devapps.diality.us/changelog/leahi-common?cs=62ce72c8cb4d90f4a0a532123b4a8f98bbca3c8d https://devapps.diality.us/changelog/leahi-common?cs=62ce72c8cb4d90f4a0a532123b4a8f98bbca3c8d Vendor - TEL - Praneeth Bunne committed 62ce72c on branch LDT-4040-rinseback, LDT-4063-power-control-and-post---fw-imp and 6 more in leahi-common LDT-4040: Merge staging into LDT-4040

· /AlarmDefs.h	(-7, +448)	| History	| Source	| Diff
· /MsgDefs.h	(-16, +84)	| History	| Source	| Diff
· /TDDefs.h	(-12, +32)	| History	| Source	| Diff

]]> Fri, 31 Jul 2026 09:33:35 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4063-1/#c30937 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4063-1/#c30937 Vendor - TEL - Arpita Srivastava commented on LEAHI-TD-FIRMWARE-LDT-4063-1

What is the use case of including this header?

What is the use case of including this header?

Should we move this up before the test functions?

Add @messages MSG_ID_TD_POST_FINAL_TEST_RESULT

Add @messages MSG_ID_TD_POST_SINGLE_TEST_RESULT

is the function description correct?
change DG -> DD

output is none

add safetyShutdownSelfTestStatus to outputs

why is this line commented out? Should we remove it or add a TODO?

Remove blank line

align comment

break this line

add @messages MSG_ID_TD_UI_GENERIC_CONFIRMATION_REQUEST

add @messages MSG_ID_TD_UI_GENERIC_CONFIRMATION_REQUEST

remove blank line

Optional: should we rename these to dd and update the comment as well.
dgPOSTPassed -> ddPOSTPassed
dgPOSTResultReceived -> ddPOSTResultReceived
dgVersion -> ddVersion

add @alarms

break line

add @alarms

Should we use the test config TEST_CONFIG_DISABLE_ALARM_AUDIO?

align comments

Add @alarms

will create a branch for dialin and make the necessary changes. Before merging will talk to Caitlynn to change the variable in automated testing as well.

Remove empty line