This is a list of all comments for HD-DEN-6078-1.

Review Summary: No summary


----------------------------------------
File: firmware/App/Controllers/Valves.c
Revision Comment by qnguyen on 15 January 2021, 09:41 
https://devapps.diality.us/cru/HD-DEN-6078-1#c7395

  Remove extra lines.

  Reply by Dara Navaei on 18 January 2021, 08:47
  > Done

    Reply by qnguyen on 18 January 2021, 10:39
    > RESOLVED in CODE WALKTHROUGH.

Revision Comment by Sean Nash on 18 January 2021, 10:03 
https://devapps.diality.us/cru/HD-DEN-6078-1#c7462

  Need an alarm (fault) to be triggered as well - otherwise f/w
  will try to continue in whatever mode we're in.

  Reply by Dara Navaei on 20 January 2021, 09:35
  > Done

    Reply by Sean Nash on 20 January 2021, 10:34
    > RESOLVED in CODE WALKTHROUGH.


---

ID: HD-DEN-6078-1 https://devapps.diality.us/cru/HD-DEN-6078-1

Title: HD-DEN-6078_Revisit Safety And Functionality of Pumps And Valves
Statement of Objectives:


State: Closed

Summary: 

Author: Dara Navaei
Reviewers: (0 active, 3 completed*)
     qnguyen   (*)
     Sean Nash   (*)
     pmontazemi   (*)