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

Review Summary: No summary


----------------------------------------
File: sources/view/settings/VHDUsageInfo.cpp
Revision Comment by Behrouz NematiPour on 22 September 2024, 23:30 
https://devapps.diality.us/cru/UI-DEN-17055-1#c20278

  Displaying hours as a float can be confusing and not
  user-friendly. 
  For example, seeing "11.7" requires users to calculate the
  actual hours and minutes, which adds unnecessary complexity. 
  Please consult with the system team to explore a more intuitive
  format for representing time, such as separating hours and
  minutes (e.g., "11h 42m" or 11:42) for better clarity.

  Reply by Dara Navaei on 23 September 2024, 10:42
  > Updated the code to not show any decimals.

    Reply by Behrouz NematiPour on 23 September 2024, 20:34
    > RESOLVED

----------------------------------------
File: sources/view/settings/VHDUsageInfo.h
Revision Comment by Behrouz NematiPour on 18 September 2024, 18:39 
https://devapps.diality.us/cru/UI-DEN-17055-1#c20275

  please use the settings configuration formatting when needed in
  the following function:
  
  {code}void View::VHDUsageInfo::onActionReceive(const
  HDUsageInfoResponseData &vData){code}
  
  **if there are other places as we talked please update them
  with that as well.**
  **please define the QSTring variable on top of the function and
  then used that variable (aka move your private variable in the
  function scope)**
  
  {code}_serviceDateFormat = _Settings.dateFormat();{code}
  
  and remove the private  variable:
  
  {code}    const QString _LastTreatmentDateFormat    =
  "yyyy/MM/dd";{code}

  Reply by Dara Navaei on 19 September 2024, 17:25
  > Done

    Reply by Behrouz NematiPour on 19 September 2024, 17:48
    > RESOLVED


---

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

Title: UI-DEN-17055_UI DN Release
Statement of Objectives:


State: Closed

Summary: 

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