This is a list of all comments for UI-DEN-17098-1.

Review Summary: No summary


----------------------------------------
File: sources/model/hd/data/MHDBloodLeakData.h
Revision Comment by Behrouz NematiPour on 24 October 2024, 19:56 
https://devapps.diality.us/cru/UI-DEN-17098-1#c20613

  Could you please search for the 0x6C00 message ID in the
  Dry-Demo and update that too.
  If you found it please do the same as we did for the 0x0900 and
  0x0B00.
  
  Also fix the dialin same as dry-demo.

  Reply by Dara Navaei on 28 October 2024, 17:31
  > Please refer to: DRY-DEMO-DEN-17098-1

    Reply by Behrouz NematiPour on 08 November 2024, 10:08
    > RESOLVED

----------------------------------------
File: sources/view/hd/data/treatment/VHDTreatmentStatesData.h
Revision Comment by Behrouz NematiPour on 24 October 2024, 19:55 
https://devapps.diality.us/cru/UI-DEN-17098-1#c20612

  Don't we need to add the new field in its Model?

  Reply by Dara Navaei on 27 October 2024, 14:02
  > No we do not need this code. I removed it.

    Reply by Behrouz NematiPour on 28 October 2024, 14:41
    > RESOLVED

----------------------------------------
File: sources/model/hd/data/treatment/MTreatmentSalineData.h
Revision Comment by Behrouz NematiPour on 28 October 2024, 14:41 
https://devapps.diality.us/cru/UI-DEN-17098-1#c20641

  fix the numbering (should be 4).

  Reply by Dara Navaei on 28 October 2024, 16:30
  > Done

    Reply by Behrouz NematiPour on 08 November 2024, 10:17
    > RESOLVED

Revision Comment by Behrouz NematiPour on 28 October 2024, 14:44 
https://devapps.diality.us/cru/UI-DEN-17098-1#c20643

  please take a look at dry-demo, as well.

  Reply by Dara Navaei on 28 October 2024, 17:31
  > Please refer to: DRY-DEMO-DEN-17098-1

    Reply by Behrouz NematiPour on 08 November 2024, 10:17
    > RESOLVED


---

ID: UI-DEN-17098-1 https://devapps.diality.us/cru/UI-DEN-17098-1

Title: UI-DEN-17098_Release 18 Changes
Statement of Objectives:


State: Closed

Summary: 

Author: Dara Navaei
Moderator: Dara Navaei
Reviewers: (3 active, 2 completed*)
     Sean Nash   (*)
     Behrouz NematiPour   (*)
     Tiffany Mejia
     jpaguio
     Michael Garthwaite