DG-DEN-16674

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

Done

Done

Can we bring a parenthesis for better readability?

Can we bring a parenthesis for better readability?

DG-DEN-16674_Td2 Calibration Feature
DG-DEN-16674_Td2 Calibration Feature
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done

Done

But this comment also asked that you remove the transition to fill mode here and add it to the fill mode function called above. It's still here.

But this comment also asked that you remove the transition to fill mode here and add it to the fill mode function called above. It's still here.

No I will check the Dialin check in the function like the rest of the commands from Dialin.

No I will check the Dialin check in the function like the rest of the commands from Dialin.

I don't see change. Did you push?

I don't see change. Did you push?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done

Done

Done

Done

Done

Done

Done. Added a check in the function in the Mode Fill like the rest of the modes and drivers.

Done. Added a check in the function in the Mode Fill like the rest of the modes and drivers.

Done

Done

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Do this (request fill mode) in the testSetModeFillForCal() function above (and only if Dialin is logged in).

Do this (request fill mode) in the testSetModeFillForCal() function above (and only if Dialin is logged in).

Make sure Dialin is logged in before accepting command.

Make sure Dialin is logged in before accepting command.

If this is a Dialin command, make sure Dialin is logged in before acting on it (throughout this module).

If this is a Dialin command, make sure Dialin is logged in before acting on it (throughout this module).

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

If these are Dialin commands, add a condition around this switch that Dialin has logged in.

If these are Dialin commands, add a condition around this switch that Dialin has logged in.

These are Dialin commands? If so, prefix each enum with CMD_ and clarify this in comment.

These are Dialin commands? If so, prefix each enum with CMD_ and clarify this in comment.