LEAHI-DD-FIRMWARE-LDT-4049_Standby - FW Implementation - 2/2:

Activity

LEAHI-DD-FIRMWARE-LDT-4049-1 6

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 3m 2 My understanding is that Standby should stop all pumps an...
    Reviewer - 1% reviewed 15m    
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed 0m    
    Reviewer - 13% reviewed 38m 4 you need to bring in the new set of valves here as well.
    Total   6h 57m 6  
    #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/Heaters.h Changed
    Open in IDE #permalink
    /firmware/App/.../FPModes/FPModeStandby.c Changed
    Open in IDE #permalink
    /firmware/App/Modes/ModeFault.c Changed 3
    Open in IDE #permalink
    /firmware/App/Modes/ModeStandby.c Changed 3
    Open in IDE #permalink
    /firmware/App/Services/Messaging.c Changed
    Open in IDE #permalink
    /firmware/App/Services/TDInterface.c Changed
    Open in IDE #permalink
    /firmware/App/Services/TDInterface.h Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time