Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge branch 'staging' into DEN-14559-dg_hd_dev-hd_dg_dvt-update-part-8

    • -13
    • +13
    ./Controllers/TemperatureSensors.c
DEN-14559 updated the code to be able request POST by the Dialin user. Updated the chemical disinfect flush to be able drain the reservoirs all the time

    • -23
    • +22
    ./Modes/ModeChemicalDisinfectFlush.c
Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Merge branch 'DEN-13946-certain-temperature-sensors-trigger-wrong-alarm' into develop

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

DEN-13946 Code review correction

Seems like there are several variables missing.

Seems like there are several variables missing.

DG-DEN-14604_DG HD Dev HD DG Dvt Update Part 9
DG-DEN-14604_DG HD Dev HD DG Dvt Update Part 9
Updated comment

Updated comment

Removed

Removed

DEN-13676 Correction per code review

DEN-13946 Corrections per code review

    • -22
    • +23
    ./Controllers/TemperatureSensors.h
This looks strange. Why is TD2 temp usually 0.0, but read if using TPo?

This looks strange. Why is TD2 temp usually 0.0, but read if using TPo?

Prefer doing this kind of thing with a loop iterating through the enum in case we add/remove a reservoir.

Prefer doing this kind of thing with a loop iterating through the enum in case we add/remove a reservoir.

Does hasAlarmBeenTriggered need to be set here too? If not, need to clarify what that flag really means.

Does hasAlarmBeenTriggered need to be set here too? If not, need to clarify what that flag really means.