LEAHI-APPLICATION-LDT-4852_SW Misc & Enhc - SW - 01 - Remove Old Messaging...

Activity

LEAHI-APPLICATION-LDT-4852-1 17

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 1h 26m 17 BN approves
    Reviewer - 0% reviewed      
    Reviewer - 100% reviewed 1m    
    Reviewer - Complete 0m    
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed 0m    
    Total   1h 28m 17  
    #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

    Nicholas Ramirez

    add plugin for reading the checkin from UI/TD 1. Show SW -> FW checkin in som...

    add plugin for reading the checkin from UI/TD
    1. Show SW -> FW checkin in some list appending or count

    • make the checkin enabled manually in CAN.py handle_messages()


    2. Show on SIM when SW is not running - SW/FW Commincaiton failure

    On application
    1. When SIM is killed app shows alarm 100

    Nicholas Ramirez

    will do in later branch

    will do in later branch

    Nicholas Ramirez

    BN approves

    BN approves

    /sources/canbus/MessageDispatcher.cpp Changed
    Open in IDE #permalink
    /sources/canbus/MessageDispatcher.h Changed
    Open in IDE #permalink
    /sources/canbus/MessageGlobals.h Changed
    Open in IDE #permalink
    /sources/canbus/MessageInterpreter.cpp Changed
    Open in IDE #permalink
    /sources/canbus/MessageInterpreter.h Changed
    Open in IDE #permalink
    /sources/gui/.../diagnostics/DiagnosticsDD.qml Changed
    Open in IDE #permalink
    /sources/gui/.../diagnostics/DiagnosticsFP.qml Changed
    Open in IDE #permalink
    /sources/gui/.../diagnostics/DiagnosticsTD.qml Changed 3
    Open in IDE #permalink
    /sources/gui/.../settings/SettingsStack.qml Changed
    Open in IDE #permalink
    /sources/gui/qml/PowerItem.qml Changed
    Open in IDE #permalink
    /sources/gui/GuiController.cpp Changed 6
    Open in IDE #permalink
    /sources/gui/GuiController.h Changed
    Open in IDE #permalink
    /sources/gui/GuiGlobals.h Changed
    Open in IDE #permalink
    /sources/gui/GuiView.cpp Changed
    Open in IDE #permalink
    /sources/gui/GuiView.h Changed
    Open in IDE #permalink
    /sources/.../treatment/MTreatmentAdjustRequests.h Changed
    Open in IDE #permalink
    /sources/model/.../alarm/MAlarmTriggered.cpp Changed
    Open in IDE #permalink
    /sources/model/hd/alarm/MAlarmTriggered.h Changed
    Open in IDE #permalink
    /sources/model/MModel.h Changed
    Open in IDE #permalink
    /sources/ApplicationController.cpp Changed 5
    Open in IDE #permalink
    /sources/ApplicationController.h Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time