This is a list of all comments for HD-DEN-16693-2.

Review Summary: No summary


----------------------------------------
File: firmware/App/Controllers/AirTrap.c
Revision Comment by Vinayakam Mani on 14 March 2024, 17:06 
https://devapps.diality.us/cru/HD-DEN-16693-2#c19622

  lets keep "///" comment for doxygen generation.

  Reply by Dara Navaei on 14 March 2024, 17:16
  > Done

    Reply by Vinayakam Mani on 14 March 2024, 17:23
    > RESOLVED IN CODEWALKTHROUGH

----------------------------------------
File: firmware/App/Controllers/PresOccl.c
Revision Comment by Vinayakam Mani on 14 March 2024, 17:08 
https://devapps.diality.us/cru/HD-DEN-16693-2#c19623

  Let's keep "///" 3 line comment for doxygen generation.

  Reply by Dara Navaei on 14 March 2024, 17:17
  > Done

    Reply by Vinayakam Mani on 14 March 2024, 17:24
    > RESOLVED IN CODE WALKTHROUGH.

----------------------------------------
File: firmware/App/Modes/ModeTreatment.c
Revision Comment by Sean Nash on 14 March 2024, 17:16 
https://devapps.diality.us/cru/HD-DEN-16693-2#c19624

  I thought we weren't going to do this anymore?

  Reply by Dara Navaei on 14 March 2024, 17:17
  > This is in a different function. I can remove.

    Reply by Sean Nash on 14 March 2024, 17:20
    > RESOLVED in CODE WALKTHROUGH.


---

ID: HD-DEN-16693-2 https://devapps.diality.us/cru/HD-DEN-16693-2

Title: HD-DEN-16693_Institutional Settings Implementation
Statement of Objectives:


State: Closed

Summary: 

Author: Dara Navaei
Moderator: Dara Navaei
Reviewers: (2 active, 2 completed*)
     Sean Nash   (*)
     Vinayakam Mani   (*)
     jpaguio
     Michael Garthwaite