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

Review Summary: No summary


----------------------------------------
File: firmware/App/Modes/ModePostTreat.c
Revision Comment by wbracken on 14 June 2023, 15:23 
https://devapps.diality.us/cru/HD-DEN-15436-1#c17890

  Add patientDisconnectHandled to header

  Reply by Darren Cox on 14 June 2023, 16:06
  > Updated.

    Reply by wbracken on 15 June 2023, 10:25
    > RESOLVED IN CODE WALKTHROUGH

Revision Comment by Sean Nash on 14 June 2023, 15:55 
https://devapps.diality.us/cru/HD-DEN-15436-1#c17891

  Do we still need this condition?  Door closed requirement may
  have been released in prior state.  Doesn't seem like this is
  truly needed to move on from here.

  Reply by Darren Cox on 14 June 2023, 16:17
  > Not required. checkPostTreatmentPatientDisconnect() releases
  > the door closed requirements when user confirms patient
  > disconnect.  Removed.

    Reply by Sean Nash on 14 June 2023, 16:50
    > CLOSED in CODE WALKTHROUGH.

      Reply by Dara Navaei on 19 October 2023, 08:59
      > RESOLVED in CODE WALKTHROUGH

Revision Comment by Sean Nash on 14 June 2023, 15:58 
https://devapps.diality.us/cru/HD-DEN-15436-1#c17892

  I feel like this whole state should be removed.  We are no
  longer doing de-prime.     Removing state will mean Dialin must
  be updated to reflect new state list.

  Reply by Darren Cox on 14 June 2023, 16:07
  > I left it because DG modes are checked and thought it must be
  > waiting for DG mode to sync before moving on.  If not an
  > issue, I can remove it.

    Reply by Sean Nash on 14 June 2023, 16:11
    > The next state - which should now be the first drain state
    > - also checks DG modes.  I don't see an issue with removing
    > this state except that next state needs to check for DG in
    > fault mode so that it will skip past drain state.

      Reply by Darren Cox on 14 June 2023, 17:38
      > State removed.

        Reply by Sean Nash on 15 June 2023, 09:22
        > RESOLVED in CODE WALKTHROUGH.


---

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

Title: HD-DEN-15436_Dpi Tubing Comes Out OF Pump Track During Dpi Reversal IN Post Treatment
Statement of Objectives:


State: Closed

Summary: 

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