· /alarmMapping.sh	(-7, +6)	| History	| Source	| Diff
· /cppcheck.sh	(-6, +6)	| History	| Source	| Diff
· /cr_objectives.sh	(-0, +15)	| History	| Source	| Diff
· /main.cpp	(-6, +6)	| History	| Source	| Diff
· /remove-exra-emptylines.sh	(-6, +6)	| History	| Source	| Diff
· /scripts/autostart	(-6, +6)	| History	| Source	| Diff
· /scripts/bluetooth_paired_clear.sh	(-6, +6)	| History	| Source	| Diff
· /scripts/bluetooth_paired_query.sh	(-6, +6)	| History	| Source	| Diff
· /scripts/brightness_get.sh	(-0, +15)	| History	| Source	| Diff
· /scripts/brightness_set.sh	(-0, +15)	| History	| Source	| Diff
· /scripts/date_time_set.sh	(-0, +15)	| History	| Source	| Diff
· /scripts/export_logs.sh	(-0, +15)	| History	| Source	| Diff
· /scripts/run.sh	(-6, +6)	| History	| Source	| Diff
· /scripts/setup.sh	(-6, +6)	| History	| Source	| Diff
· /scripts/start.sh	(-6, +6)	| History	| Source	| Diff
… 570 more files in changeset.

]]> Thu, 29 Apr 2021 09:44:53 -0700 Behrouz NematiPour https://devapps.diality.us/cru/UI-DEN-7820-1 https://devapps.diality.us/cru/UI-DEN-7820-1 Behrouz NematiPour Closed UI-DEN-7820-1 ]]> Tue, 27 Apr 2021 10:51:03 -0700 https://devapps.diality.us/cru/UI-DEN-7820-1 https://devapps.diality.us/cru/UI-DEN-7820-1 plucia  (deleted user) and pmontazemi  (deleted user) finished reviewing UI-DEN-7820-1 UI-DEN-7820_UI DEV Post Treatment ]]> Tue, 27 Apr 2021 10:49:43 -0700 plucia https://devapps.diality.us/cru/UI-DEN-7820-1/#c9635 https://devapps.diality.us/cru/UI-DEN-7820-1/#c9635 plucia  (deleted user) replied to Behrouz NematiPour in UI-DEN-7820-1

RESOLVED

Updated.

Looks like this needs to be updated

RESOLVED.

That is the function signature and doesn't have to have any parameter defined if it is not being used and no argument is passing to it.
This way we don't need to use the Q_UNUSED(vData); anymore for parameters not being used.

Dereferencing which parameter here? (sorry I have never seen this in the past)

· /denali.pro	(-143, +174)	| History	| Source	| Diff
· /denali.pro.user	(-1, +1)	| History	| Source	| Diff
· /denali.qrc	(-18, +18)	| History	| Source	| Diff
· /en_US.udic	(-0, +11)	| History	| Source	| Diff
· /sources/canbus/MessageDispatcher.cpp	(-0, +43)	| History	| Source	| Diff
· /sources/canbus/MessageDispatcher.h	(-25, +29)	| History	| Source	| Diff
· /sources/canbus/MessageGlobals.h	(-91, +103)	| History	| Source	| Diff
· /sources/canbus/MessageInterpreter.cpp	(-72, +86)	| History	| Source	| Diff
· /sources/gui/GuiGlobals.cpp	(-0, +5)	| History	| Source	| Diff
· /sources/gui/GuiGlobals.h	(-134, +152)	| History	| Source	| Diff
· /sources/gui/qml/components/ImageText.qml	(-1, +1)	| History	| Source	| Diff
· /sources/gui/qml/main.qml	(-107, +117)	| History	| Source	| Diff
· /sources/gui/qml/pages/MainStack.qml	(-10, +8)	| History	| Source	| Diff
· /sources/gui/qml/pages/endtreatment/EndTreatmentEndStack.qml	(-0, +48)	| History	| Source
· /sources/gui/qml/pages/endtreatment/EndTreatmentRecirculateStack.qml	(-0, +57)	| History	| Source
… 319 more files in changeset.

]]> Tue, 16 Feb 2021 16:23:46 -0800 Behrouz NematiPour https://devapps.diality.us/cru/UI-DEN-6631-1 https://devapps.diality.us/cru/UI-DEN-6631-1 Behrouz NematiPour Closed UI-DEN-6631-1 ]]> Tue, 16 Feb 2021 16:09:41 -0800 https://devapps.diality.us/cru/UI-DEN-6631-1 https://devapps.diality.us/cru/UI-DEN-6631-1 plucia  (deleted user) and pmontazemi  (deleted user) finished reviewing UI-DEN-6631-1 UI-DEN-6631_UI DEV POST Treatment Rinseback ]]> Tue, 16 Feb 2021 13:49:16 -0800 plucia https://devapps.diality.us/cru/UI-DEN-6631-1/#c8157 https://devapps.diality.us/cru/UI-DEN-6631-1/#c8157 plucia  (deleted user) replied to Behrouz NematiPour in UI-DEN-6631-1

RESOLVED

RESOLVED

Removed.

Thanks for catching that,
Fixed.

I think you meant fontsPixelCreateTreatmentTable

These are currently unused, maybe they can be deleted?

RESOLVED.

RESOLVED.

RESOLVED.

it is a kind of hierarchy commenting that the //// is the title and the rest beneath that are like 1.1 and 1.2 so it helps to organize better.

Only two "/" for comments in QML.

RESOLVED.

Removed.

Removed.

fixed.

Remove perhaps which suggest incertitude in Class B code.

it is as formal (instead of it's as informal)