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

Review Summary: No summary


----------------------------------------
File: firmware/App/Modes/PreTreatmentRecirc.c
Revision Comment by wbracken on 06 February 2023, 13:50 
https://devapps.diality.us/cru/HD-DEN-14547-1#c16378

  Alphabetical ordering.

  Reply by Michael Garthwaite on 06 February 2023, 14:12
  > it was a double include anyway. Fixed. Thanks!

    Reply by wbracken on 06 February 2023, 16:26
    > RESOLVED IN CODE WALKTHROUGH.

Revision Comment by jtaylor on 07 February 2023, 15:08 
https://devapps.diality.us/cru/HD-DEN-14547-1#c16394

  Comment Alignment

  Reply by Michael Garthwaite on 07 February 2023, 16:52
  > Fixed. Thanks!

    Reply by jtaylor on 08 February 2023, 16:50
    > RESOLVED IN CODE WALKTHROUGH.

Revision Comment by Darren Cox on 07 February 2023, 11:58 
https://devapps.diality.us/cru/HD-DEN-14547-1#c16389

  Use SEC_PER_MIN instead of 60.  Also comment says min, but
  really it is ms.

  Reply by Michael Garthwaite on 07 February 2023, 13:53
  > Fixed. Thanks!

    Reply by Darren Cox on 08 February 2023, 16:53
    > RESOLVED IN CODE WALKTHROUGH.


---

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

Title: HD-DEN-14547_Incorrect DG Event
Statement of Objectives:


State: Closed

Summary: 

Author: Michael Garthwaite
Moderator: Michael Garthwaite
Reviewers: (5 active, 1 completed*)
     Darren Cox   (*)
     Sean Nash
     wbracken
     Dara Navaei
     jtaylor
     Steve Jarpe