Services

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-15199 addressed the code review comments

Done

Done

Done

Done

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done

Done

Done.

Done.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Brought the code back.

Brought the code back.

Why did we blank title/instructions?

Why did we blank title/instructions?

Acking even if payload length > 0?

Acking even if payload length > 0?

Checking for isTestingActivated should be done in sendTestConfigStatusToDialin function (not here). Fix throughout new handlers.

Checking for isTestingActivated should be done in sendTestConfigStatusToDialin function (not here). Fix throughout new handlers.

This is an output.

This is an output.

Remove "Been" from function name. And add a comment line above explaining what we're doing here.

Remove "Been" from function name. And add a comment line above explaining what we're doing here.

DEN-15199 added to code to check for dialin check in

  1. … 3 more files in changeset.
Done

Done

Done

Done

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.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Not a Dialin command. Called by opmodes in firmware.

Not a Dialin command. Called by opmodes in firmware.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Yes, as this function could be called from somewhere other than message handler. RESOLVED in CODE WALKTHROUGH.

Yes, as this function could be called from somewhere other than message handler.
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Still want check for logged in.

Still want check for logged in.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DEN-15199 addressed the code review comments

  1. … 1 more file in changeset.