UI-DEN-10424_FW Request A4 A5 Msg Log

Activity

UI-DEN-10424-1 3

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 16m 1 Since we have more than one message which is using the dy...
    Reviewer - Complete 9m    
    pmontazemi  (deleted user)
    Reviewer completed
    Reviewer - Complete 13m 2 RESOLVED.
    Total   39m 3  
    #permalink

    Objectives

    commit : 161d1431 - DEN-9968: GUI does not indicate the amount of time the device has been stopped in pre-treatment & treatment

    • Added message ID_TreatmentStop = 0x4900, // 73 and its Model, View
    • Added a timer on the Alarm Dialog which is visible in Treatment Stop and if the message above has a total value.
      commit : b94c297f - Part 2/2: Added the two new General HD/DG messages



    commit : 1f89cfbb - DEN-10161: Missing END TREATMENT option in Treatment-Recirculate-Stop screen

    • The screen (Reconnect) belongs to the TREATMENT_RECIRC_STOPPED_STATE state of the TREATMENT_RECIRC_STATE in the TREATMENT_STATE message, didn't even have a "END TREATMENT" button, which has been added.
    • The time_IsLeft logic has NOT been removed yet, since I don't recall why it has been used in the first place.
      commit : 338575f7 - Part 2/2: make the Alarm Triggered messages modification

    • Created a class named MAbstractDynamic which is the base class of the dynamic model classes.
      + Dynamic Model classes are the ones with field descriptor (FD) and a data field (DF) after that which the data type in the data field is defined by field descriptor.
      + This combination for now is only for 2 set of FD, DF only.
    • Made the Alarm Triggered inherit from MAbstractDynamic.
      commit : 67d05211 - Updated AlarmMapping



    commit : cbb246d6 - Part 1/2: make the unhandled messages dynamic

    • The messages/unhandled.conf has been created in the settings area and will be loaded and used for the unhandled messages in the message interpreter.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /resources/images/Logo d-blue.png Added
    Open in IDE #permalink
    /resources/.../messages/unhandled.conf Added
    Open in IDE #permalink
    /sources/canbus/MessageDispatcher.cpp Changed
    /sources/canbus/MessageDispatcher.h Changed
    /sources/canbus/MessageGlobals.h Changed
    /sources/canbus/MessageInterpreter.cpp Changed
    /sources/canbus/MessageInterpreter.h Changed
    /sources/.../dialogs/NotificationDialog.qml Changed
    /sources/.../endtreatment/EndTreatmentRecirculateStack.qml Changed
    /sources/gui/qml/AlarmItem.qml Changed
    /sources/gui/qml/main.qml Changed
    /sources/gui/GuiGlobals.cpp Changed
    /sources/gui/GuiGlobals.h Changed
    /sources/model/dg/data/MDGGeneralEvent.cpp Added
    Open in IDE #permalink
    /sources/model/dg/data/MDGGeneralEvent.h Added
    Open in IDE #permalink
    /sources/model/hd/alarm/MAlarmMapping.cpp Changed
    /sources/model/.../alarm/MAlarmTriggered.cpp Changed 3
    /sources/model/hd/alarm/MAlarmTriggered.h Changed
    /sources/.../treatment/MTreatmentStopData.cpp Added
    Open in IDE #permalink
    /sources/.../treatment/MTreatmentStopData.h Added
    Open in IDE #permalink
    /sources/model/hd/data/MHDDebugText.cpp Changed
    /sources/model/hd/data/MHDDebugText.h Changed
    /sources/model/hd/data/MHDGeneralEvent.cpp Added
    Open in IDE #permalink
    /sources/model/hd/data/MHDGeneralEvent.h Added
    Open in IDE #permalink
    /sources/model/MAbstractDynamic.cpp Added
    /sources/model/MAbstractDynamic.h Added
    /sources/model/MModel.h Changed
    /sources/storage/StorageGlobals.cpp Changed
    /sources/.../treatment/VTreatmentStopData.cpp Added
    Open in IDE #permalink
    /sources/.../treatment/VTreatmentStopData.h Added
    Open in IDE #permalink
    /sources/view/VView.h Changed
    /sources/ApplicationController.cpp Changed
    /sources/ApplicationController.h Changed
    /AlarmMapping.csv Changed
    /denali.pro Changed
    /denali.pro.user Changed
    /en_US.udic Changed
    /ui_build_history.log Deleted
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time