Build_Issue_Merging__DEN-12716-UI_BN_S70__to__staging

Activity

UI-DEN-13726-2 0

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 10m    
    Reviewer - 0% reviewed 1m    
    Reviewer - Complete 2m    
    Reviewer - Complete 4m    
    Reviewer - Complete 11m    
    Total   30m 0  
    #permalink

    Objectives

    after the code review completion of the branch S70, it couldn't be merged into the staging due to some changes on the "common" repository.
    Those issues have been fixed to be able to merge.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /sources/.../pretreatment/PreTreatmentPrimeStack.qml Changed
    /sources/gui/qml/pages/MainStack.qml Changed
    /sources/gui/GuiGlobals.h Changed
    /sources/.../treatment/MTreatmentAdjustUltrafiltrationStateResponse.cpp Changed
    /sources/.../treatment/MTreatmentAdjustUltrafiltrationStateResponse.h Changed
    /sources/model/hd/alarm/MAlarmMapping.cpp Changed
    /sources/.../treatment/VTreatmentAdjustmentUltrafiltrationState.cpp Changed
    /sources/.../pretreatment/VPreTreatmentStatesData.cpp Changed
    /sources/.../pretreatment/VPreTreatmentStatesData.h Changed
    /sources/.../treatment/VHDTreatmentStatesData.cpp Changed
    /sources/.../treatment/VHDTreatmentStatesData.h Changed
    /unittests/tst_models.cpp Changed
    /AlarmMapping.csv Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time