DIALIN-DEN-8514_UI DEV Alarms Active List Merge Setting

Activity

DIALIN-DEN-8514-2 6

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 2h 5m 2 That function doesn't use the UI message Id and is usin...
    plucia  (deleted user)
    Reviewer completed
    Reviewer - Complete 22m 2 RESOLVED
    pmontazemi  (deleted user)
    Reviewer completed
    Reviewer - Complete 6m 2 RESOLVED.
    Total   2h 34m 6  
    #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

    There are no general comments on this review.
    /dialin/common/__init__.py Changed
    /dialin/common/hd_defs.py Changed
    Open in IDE #permalink
    /dialin/common/msg_defs.py Changed
    /dialin/protocols/CAN.py Changed 3
    /dialin/ui/__init__.py Changed
    Open in IDE #permalink
    /dialin/ui/dg_simulator.py Added 3
    /dialin/ui/hd_simulator.py Changed
    /dialin/ui/hd_simulator_alarms.py Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time