•  

Comment Results

Review Name Created Custom Fields Content
DG-DEN-14497-1 11 Jan 2023

RESOLVED in CODE WALKTHROUGH.

LEAHI-APPLICATION-LDT-2160-1 05 Sep 2025

remove Image componenet

DG-DEN-14497-1 11 Jan 2023

RESOLVED in CODE WALKTHROUGH.

DG-DEN-14497-1 11 Jan 2023

RESOLVED in CODE WALKTHROUGH.

LEAHI-TD-FIRMWARE-LDT-1903-8 25 Feb 2026

it is used for publish function in the bottom of this function publishAirPumpData();

DG-DEN-14497-1 11 Jan 2023

RESOLVED in CODE WALKTHROUGH.

LEAHI-APPLICATION-LDT-1021-1 26 Jun 2025

RESOLVED

DG-DEN-14497-1 11 Jan 2023

RESOLVED in CODE WALKTHROUGH.

LEAHI-DD-FIRMWARE-LDT-2030-2 12 Feb 2026

done

LEAHI-DD-FIRMWARE-LDT-2198-1 15 Sep 2025

fixed. thanks

DG-DEN-14497-1 11 Jan 2023

RESOLVED IN CODE WALKTHROUGH.

DG-DEN-14497-1 11 Jan 2023

RESOLVED IN CODE WALKTHROUGH.

LEAHI-APPLICATION-LDT-2160-1 05 Sep 2025

Is there a reason we cannot merge the two PreTreatmentCreate.qml and PreTreatmentCreateContent.qml in one PreTreatmentCreate.qml file?

DIALIN-DEN-14497-1 11 Jan 2023

Done

LEAHI-APPLICATION-LDT-1021-1 26 Jun 2025

Please always use spaces before and after the not ( "!" ) character.

DIALIN-DEN-14497-1 11 Jan 2023

Done

RO-LDT-566-1 27 Jun 2025

Should we update group names (RO to FP)?

DIALIN-DEN-14497-1 11 Jan 2023

Resolved.

LEAHI-APPLICATION-LDT-853-1 26 Jun 2025

this is not the place to define the content of the headerbar as we discussed.
if possible keep the code in your tmp or any....

LEAHI-APPLICATION-LDT-2160-1 05 Sep 2025

when value changed when value is less than minimum set to minimum

LEAHI-APPLICATION-LDT-1788-1 13 Aug 2025

File Renamed and logic updated.
RESOLVED.

DD-LDT-1873-1 13 Aug 2025

Done.

LEAHI-APPLICATION-LDT-1788-1 13 Aug 2025

RESOLVED.

LEAHI-APPLICATION-LDT-2160-1 15 Sep 2025

RESOLVED

LEAHI-APPLICATION-LDT-1788-1 13 Aug 2025

For now we can use the pointer, and we will investigate on this.
RESOLVED.

LEAHI-APPLICATION-LDT-2160-1 05 Sep 2025

update to clear after validation fail when you re edit

LEAHI-TD-FIRMWARE-LDT-2749-1 12 Feb 2026

updated.

LEAHI-APPLICATION-LDT-2160-1 15 Sep 2025

RESOLVED

LEAHI-APPLICATION-LDT-2160-1 15 Sep 2025

RESOLVED

LEAHI-APPLICATION-LDT-1596-1 13 Aug 2025

i kept the default the same as it was so other components will not be changed

LEAHI-APPLICATION-LDT-1788-1 13 Aug 2025

updated

LEAHI-APPLICATION-LDT-1788-1 13 Aug 2025

Remove leading and trailing spaces in if conditionals to keep style consistent.

LEAHI-APPLICATION-LDT-2160-1 05 Sep 2025

removed

LEAHI-TD-FIRMWARE-LDT-1840-1 12 Feb 2026

Removed

LEAHI-DD-FIRMWARE-LDT-3344-1 12 Feb 2026

Use #define macros for magic numbers.

LEAHI-TD-FIRMWARE-LDT-3126-1 26 Feb 2026

We do not need to set result here since it is already set to run state at top of this function.

LEAHI-TESTSUITES-LDT-2741-1 13 Feb 2026

I don't think anything other than the container, and [ id or objectName ] should be in the object definition dictionary.
If, as an example, the passwordCharacter is changed to #, the object cannot be found. same for echoMode.

LEAHI-TD-FIRMWARE-LDT-1903-8 27 Feb 2026

Alarm should be handled inside of setAirPumpState().

LEAHI-BOOTLOADER-LDT-3259-1 27 Feb 2026

else shouldn't we go back to waiting for an update or something? Seems wrong to just get stuck here.

LEAHI-TD-FIRMWARE-LDT-3126-1 27 Feb 2026

Added

LEAHI-TD-FIRMWARE-LDT-3126-1 27 Feb 2026

Added

LEAHI-TD-FIRMWARE-LDT-1903-8 26 Feb 2026

Add blank line after declarations.

LEAHI-TD-FIRMWARE-LDT-3126-1 26 Feb 2026

Should be only one blank line here.

LEAHI-TD-FIRMWARE-LDT-3126-1 26 Feb 2026

Remove TODO comment since you have done it.

DG-DEN-14497-1 11 Jan 2023

RESOLVED in CODE WALKTHROUGH.

DG-DEN-14497-1 10 Jan 2023

What is difference between P coefficient and P term a few lines above?

DIALIN-DEN-14497-1 11 Jan 2023

No this is command schedules the entire zero command sequence while the other embedded mode commands are atomic.

DG-DEN-14706-2 31 Aug 2023

Too many () after &&?

LEAHI-DIALIN-LEAH-136-3 22 Jan 2025

Rename these per f/w refactoring changes.

DG-DEN-14706-2 01 Sep 2023

RESOLVED in CODE WALKTHROUGH.