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

Review Summary: No summary


----------------------------------------
File: sources/applicationcontroller.h
Revision Comment by lbaloa on 24 October 2019, 09:14 
https://devapps.diality.us/cru/UI-DEN-626-1#c123

  initialize with nullptr for consistency

  Reply by Behrouz NematiPour on 24 October 2019, 10:46
  > Thanks for the comment.
  > 
  > static class member variable can not be initialized in
  > header.
  > Please take a look at the applicationcontroller.cpp@21.
  > It has been initialized with nullptr.
  > 
  > Note : To see the whole file content on the top right click
  > on the "View" menu and select "Full Context"

  Reply by Behrouz NematiPour on 28 October 2019, 17:47
  > seems like when I replied the emails were not working
  > Just as a reminder.

  Reply by pmontazemi on 01 November 2019, 15:34
  > Leo,
  > 
  > Resolving the case in not enough, you also need to reply to
  > Behrouz's last comment and mention the word "RESOLVED",
  > otherwise, Dara's Crucible script won't pick this code review
  > up. We will review this next week during the training.

  Reply by lbaloa on 04 December 2019, 08:51
  > RESOLVED


---

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

Title: UI-DEN-626_UI Power Off Screen
Statement of Objectives:


State: Closed

Summary: 

Author: Behrouz NematiPour
Moderator: Behrouz NematiPour
Reviewers: (0 active, 3 completed*)
     Dara Navaei   (*)
     pmontazemi   (*)
     lbaloa   (*)