dgfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-8103: Enable water quality checks

DEN-7605: Added tare state to drain mode to allow time for reservoir to settle first.

    • -7
    • +33
    /firmware/App/Services/Reservoirs.c
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DEN-7605: Addressed code review feedback.

Fixed.

Fixed.

Fixed.

Fixed.

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

Fixed merge conflicts.

    • -4
    • +109
    /firmware/App/Controllers/DrainPump.c
    • -1
    • +4
    /firmware/App/Modes/ModeHeatDisinfect.c
DEN-7605: Added delays to drain mode.

    • -0
    • +12
    /firmware/App/Controllers/DrainPump.c
DEN-8103: Imposed maximum crc size

    • -101
    • +2
    /firmware/App/Services/AlarmMgmt.h
    • -0
    • +130
    /firmware/App/Services/AlarmMgmtSWFaults.h
    • -25
    • +54
    /firmware/App/Services/Integrity.c
The only preference is to put TRUE first if it is == such as below.

The only preference is to put TRUE first if it is == such as below.

I have no preference on that. Data is temporary either way. By reference is a little easier on the stack.

I have no preference on that. Data is temporary either way. By reference is a little easier on the stack.

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

Should we move this enum to separate .h file?

Should we move this enum to separate .h file?

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

Place TRUE as first argument.

Place TRUE as first argument.

If we align, we align everything, otherwise, no extra spaces between variables and the "=" sign.

If we align, we align everything, otherwise, no extra spaces between variables and the "=" sign.

Align comment.

Align comment.

DEN-8103: Alarm when firmware image integrity check failed

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

DEN-7605: Added options to valves and pumps to delay changes in state.

    • -4
    • +97
    /firmware/App/Controllers/DrainPump.c
DG-DEN-8103_DG HD Dev Alarm Design
DG-DEN-8103_DG HD Dev Alarm Design
DEN-7605: Initialize fluid leak override structure on power up.

I thought we decided to pass the broadcast data as a reference. Did we change that?

I thought we decided to pass the broadcast data as a reference. Did we change that?

Merged DEN-8030_chemical disinfect

DEN-8030 fixed load cells tare

Removed casting.

Removed casting.

DEN-7802: Added fill mode paused state

The macro already cast the data to U32.

The macro already cast the data to U32.

Should maybe cast here though.

Should maybe cast here though.