This is a list of all comments for UI-DEN-16123-5.

Review Summary: No summary


----------------------------------------
File: sources/cloudsync/CloudSyncController.cpp
Revision Comment by vduong on 14 August 2023, 08:40 
https://devapps.diality.us/cru/UI-DEN-16123-5#c18567

  Did you purposely leave this commented out?

  Reply by Behrouz NematiPour on 14 August 2023, 11:41
  > save as the other one.

    Reply by vduong on 14 August 2023, 11:56
    > RESOLVED

Revision Comment by vduong on 14 August 2023, 08:40 
https://devapps.diality.us/cru/UI-DEN-16123-5#c18568

  Did you purposely leave this commented out?

  Reply by Behrouz NematiPour on 14 August 2023, 11:40
  > Yes that was intentional.
  > I don't want to expose what CRC is expected.
  > And could not find a good way to tag it as // DEBUG

    Reply by vduong on 14 August 2023, 11:56
    > RESOLVED

Revision Comment by msuleiman on 14 August 2023, 10:25 
https://devapps.diality.us/cru/UI-DEN-16123-5#c18570

  Missing semicolon ; at end

  Reply by Behrouz NematiPour on 14 August 2023, 18:21
  > This does not need  a semicolon.
  > RESOLVED

Revision Comment by vduong on 14 August 2023, 08:41 
https://devapps.diality.us/cru/UI-DEN-16123-5#c18569

  This one really needs the doxygen comment for documentation
  purposes

  Reply by Behrouz NematiPour on 14 August 2023, 13:19
  > Sure,
  > Added

    Reply by vduong on 14 August 2023, 13:19
    > RESOLVED.


---

ID: UI-DEN-16123-5 https://devapps.diality.us/cru/UI-DEN-16123-5

Title: DEN-16123-UI-BN-S103-CSFR-TokenRemoval	
Statement of Objectives:


State: Closed

Summary: 

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