This is a list of all comments for DIALIN-DEN-15014-1.

Review Summary: No summary


----------------------------------------
File: dialin/common/msg_ids.py
Revision Comment by Sean Nash on 28 March 2023, 17:35 
https://devapps.diality.us/cru/DIALIN-DEN-15014-1#c16966

  Update per f/w name change from code review.

  Reply by Dara Navaei on 28 March 2023, 21:11
  > Done

    Reply by Sean Nash on 29 March 2023, 08:07
    > RESOLVED.

----------------------------------------
File: dialin/dg/dialysate_fill.py
Revision Comment by Sean Nash on 28 March 2023, 17:36 
https://devapps.diality.us/cru/DIALIN-DEN-15014-1#c16967

  I think timestamp should go last?

  Reply by Dara Navaei on 28 March 2023, 21:06
  > Done

    Reply by Sean Nash on 29 March 2023, 08:07
    > RESOLVED.

----------------------------------------
File: dialin/hd/usage_info_record.py
Revision Comment by Sean Nash on 28 March 2023, 17:38 
https://devapps.diality.us/cru/DIALIN-DEN-15014-1#c16968

  Are these prints permanent?  May slow dialin down a bit.

  Reply by Dara Navaei on 28 March 2023, 21:06
  > They had been removed.

    Reply by Sean Nash on 29 March 2023, 08:07
    > RESOLVED.


---

ID: DIALIN-DEN-15014-1 https://devapps.diality.us/cru/DIALIN-DEN-15014-1

Title: DIALIN-DEN-15014_DG HD Dev HD DG Dvt Update Part
Statement of Objectives:


State: Closed

Summary: 

Author: Dara Navaei
Moderator: Dara Navaei
Reviewers: (4 active, 1 completed*)
     Sean Nash   (*)
     Tiffany Mejia
     Michael Garthwaite
     Behrouz NematiPour
     jishii