LEAHI-DD-FIRMWARE-LDT-2185_(IO, FP) Water Treatment and Degassing IOFP Alpha...

Activity

LEAHI-DD-FIRMWARE-LDT-2185-1 120

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author & Moderator 6h 39m 47 added param and output
    Reviewer - 0% reviewed      
    Reviewer - 96% reviewed 1h 33m 32 Update function header to include message param.
    Reviewer - 0% reviewed      
    Reviewer - 96% reviewed 2m    
    Reviewer - 3% reviewed 8m    
    Reviewer - 0% reviewed      
    Reviewer - 90% reviewed 24m    
    Reviewer - 74% reviewed 4h 59m 41 sensor name required in comments
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed      
    Total   13h 48m 120  
    #permalink

    Objectives

    There are no specific objectives for this review.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /firmware/App/Controllers/PermeateTank.c Changed 4
    /firmware/App/Controllers/ROPump.c Changed
    /firmware/App/.../FPModes/FPOperationModes.c Changed 2
    Open in IDE #permalink
    /firmware/App/.../FPModes/FlushConcentrate.c Changed 2
    /firmware/App/Modes/FPModes/FlushFilter.c Changed 6
    /firmware/.../FPModes/FlushFilterDefeatured.c Added 6
    Open in IDE #permalink
    /firmware/.../FPModes/FlushFilterDefeatured.h Added
    Open in IDE #permalink
    /firmware/App/.../FPModes/FlushPermeate.c Changed
    /firmware/App/.../FPModes/FlushPermeate.h Changed
    /firmware/App/.../FPModes/InletPressureCheck.c Changed 8
    /firmware/App/.../FPModes/InletPressureCheck.h Changed
    /firmware/App/.../FPModes/ModeGenPermeate.c Changed 12
    /firmware/.../FPModes/ModeGenPermeateDefeatured.c Changed
    /firmware/App/.../FPModes/ModePreGenPermeate.c Changed 1
    /firmware/.../FPModes/ModePreGenPermeateDefeatured.c Changed 18
    /firmware/App/Monitors/Level.h Changed
    /firmware/.../Monitors/WaterQualityMonitor.c Changed 51
    /firmware/.../Monitors/WaterQualityMonitor.h Changed 4
    /firmware/App/Services/Messaging.c Changed 4
    /AlarmDefs.h Changed
    Open in IDE #permalink
    /FPDefs.h Changed
    Open in IDE #permalink
    /MsgDefs.h Changed 2

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time