CLOUDSYNC-DEN-16690_UI MG S117

Activity

CLOUDSYNC-DEN-16690-1 26

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author & Moderator 57m 9 The exception handling was redundant since the message ha...
    Reviewer - 5% reviewed 2m    
    Reviewer - Complete 17m 1 RESOLVED on behalf of Behrouz NematiPour
    Reviewer - Complete 1h 28m 16 RESOLVED
    Reviewer - 5% reviewed 42m    
    msuleiman  (deleted user)
    Reviewer - 0% reviewed 13m    
    Reviewer - 64% reviewed 0m    
    Total   3h 41m 26  
    #permalink

    Objectives

    There are no specific objectives for this review.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    Behrouz NematiPour

    Merge branch 'DEN-16690-UI-MG-S117' into DEN-16839-UI-BN-Whitney-1B1 c88f546a...

    Merge branch 'DEN-16690-UI-MG-S117' into DEN-16839-UI-BN-Whitney-1B1
    c88f546ad09c2d6c0222d8e694a9814e37a424ea
    The follow up CR would be http://devapps.diality.us:8060/cru/CLOUDSYNC-DEN-16839-1
    Will be closed.

    /cloudsync/common/enums.py Changed
    /cloudsync/handlers/ui_cs_request_handler.py Changed 4
    Open in IDE #permalink
    /cloudsync/utils/globals.py Changed
    Open in IDE #permalink
    /cloudsync/utils/helpers.py Changed 20
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time