application

Clone Tools

Stats

Commits this week: 10
Total Committers: 14
Last Commit: 03 Apr
Commits this week: 10

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 24 May 2021
1:33 pm

Behrouz NematiPour

DEN-8504: Svc: Code Review 0

Options
    • -0
    • +1
    /resources/settings/Service.conf
    • -3
    • +3
    /sources/view/hd/data/VHDOperationModeData.cpp
11:17 am

pmontazemi  (deleted user)

remove the before anything

remove the before anything

11:16 am

pmontazemi  (deleted user)

does not (instead of doesn't)

does not (instead of doesn't)

11:16 am

pmontazemi  (deleted user)

change (instead of changed)

change (instead of changed)

11:12 am

pmontazemi  (deleted user)

Add TODO

Add TODO

9:06 am

plucia  (deleted user)

V&V has requested to be able to get the version from the UI using dialin to automate their testing

V&V has requested to be able to get the version from the UI using dialin to automate their testing

9:01 am

plucia  (deleted user)

The UI is already talking with Dialin. Your simulator is using dialin to talk to the UI. Because the UI doesn't recognize Dialin at all, Dialin has to talk on the HD to UI or DG to UI channels inst...

The UI is already talking with Dialin. Your simulator is using dialin to talk to the UI.
Because the UI doesn't recognize Dialin at all, Dialin has to talk on the HD to UI or DG to UI channels instead when sending up commands to the UI.
This poses a security risk because the UI could never restrict communication to just the DG/HD only. If Dialin can talk to the UI on separate channels then those channels can be ignored in production mode later.

It's your stack so your call but something I think we should consider

8:51 am

Behrouz NematiPour

If we decide UI to talk with Dialin we need to add Dialin channels later, but it is not the case now.

If we decide UI to talk with Dialin we need to add Dialin channels later, but it is not the case now.

8:49 am

Behrouz NematiPour

On the UI side, we have no plan, for now, to listen to Dialin.

On the UI side, we have no plan, for now, to listen to Dialin.

8:44 am

plucia  (deleted user)

These definitions force dialin to talk on an HD/DG channels in order to simulate the UI. Seems like a security risk and that the UI should be able to tell the difference.

These definitions force dialin to talk on an HD/DG channels in order to simulate the UI. Seems like a security risk and that the UI should be able to tell the difference.

8:18 am

plucia  (deleted user)

Dialin is a known CAN source.

Dialin is a known CAN source.

8:07 am

plucia  (deleted user)

The dialin usage is incorrect here. Please see my comments in the dialin code review.

The dialin usage is incorrect here. Please see my comments in the dialin code review.

7:13 am

plucia  (deleted user)

The bytearray conversions need to be inside the dialin function

The bytearray conversions need to be inside the dialin function

7:13 am

plucia  (deleted user)

The bytearray conversions need to be inside the cmd_send_post_treatment_log_response function

The bytearray conversions need to be inside the cmd_send_post_treatment_log_response function

7:00 am

plucia  (deleted user)

I assume this is just the stderr output from running alarmMapping. Is there a reason why it needs to be part of the repository?

I assume this is just the stderr output from running alarmMapping. Is there a reason why it needs to be part of the repository?

6:59 am

plucia  (deleted user)

Should the log file be added to the repository?

Should the log file be added to the repository?

6:58 am

plucia  (deleted user)

Shouldn't the source code be added to the repository instead of the binary file?

Shouldn't the source code be added to the repository instead of the binary file?

6:56 am

plucia  (deleted user)

Copyright is missing

Copyright is missing

6:56 am

plucia  (deleted user)

Copyright is missing

Copyright is missing

6:55 am

plucia  (deleted user)

The copyright is missing

The copyright is missing

6:55 am

plucia  (deleted user)

The copyright is missing

The copyright is missing

6:53 am

plucia  (deleted user)

Docstring is missing

Docstring is missing

6:52 am

plucia  (deleted user)

The copyright is missing

The copyright is missing

6:46 am

plucia  (deleted user)

The filename needs to be updated

The filename needs to be updated

6:46 am

plucia  (deleted user)

This filename needs to be updated

This filename needs to be updated

6:45 am

plucia  (deleted user)

RESOLVED

RESOLVED

Sunday 23 May 2021
8:01 pm

Behrouz NematiPour

DEN-8494: Implementation [ Service Log in & About ]

Options
    • -1
    • +1
    /cppcheck.err
    • -0
    • +2
    /denali.pro
    • -1
    • +1
    /denali.pro.user
    • -0
    • +2
    /en_US.udic
    • -0
    • +9
    /scripts/brightness_set.sh
    • -0
    • +6
    /scripts/copy.sh
    • -2
    • +18
    /sources/canbus/MessageDispatcher.cpp
    • -0
    • +1
    /sources/canbus/MessageDispatcher.h
    • -0
    • +12
    /sources/canbus/MessageGlobals.h
    • -12
    • +32
    /sources/canbus/MessageInterpreter.cpp
    • -0
    • +2
    /sources/canbus/MessageInterpreter.h
    • -0
    • +2
    /sources/gui/GuiGlobals.h
    • -2
    • +8
    /sources/gui/qml/pages/settings/SettingsDG.qml
    • -1
    • +2
    /sources/gui/qml/pages/settings/SettingsDateTime.qml
    • -1
    • +1
    /sources/model/MAbstract.h
  16. … 5 more files in changeset.
Saturday 22 May 2021
10:33 pm

Behrouz NematiPour

DEN-8494: Implementation: Merge the Service Login & About into the code.

- Added Service login

- Added having the comment capability in settings .conf files.

- Performance improvement in settings reader.

Options
    • -1
    • +1
    /denali.pro.user
    • -0
    • +5
    /resources/settings/Service.conf
    • -0
    • +2
    /sources/gui/qml/compounds/TouchGrid.qml
    • -2
    • +4
    /sources/gui/qml/pages/TreatmentFlowBase.qml
    • -1
    • +5
    /sources/gui/qml/pages/UserConfirmation.qml
    • -12
    • +15
    /sources/gui/qml/pages/settings/SettingsBase.qml
    • -2
    • +3
    /sources/gui/qml/pages/settings/SettingsDG.qml
    • -2
    • +2
    /sources/gui/qml/pages/settings/SettingsDateTime.qml
    • -52
    • +19
    /sources/gui/qml/pages/settings/SettingsHome.qml
    • -15
    • +79
    /sources/gui/qml/pages/settings/SettingsStack.qml
    • -7
    • +8
    /sources/gui/qml/pages/settings/SettingsWiFi.qml
    • -25
    • +44
    /sources/storage/Settings.cpp
    • -0
    • +5
    /sources/storage/Settings.h
    • -1
    • +2
    /sources/view/settings/VSettings.cpp
5:11 pm

Behrouz NematiPour

A little optimization

Options
    • -1
    • +1
    /denali.pro.user
    • -2
    • +2
    /sources/gui/qml/pages/MainStack.qml
    • -4
    • +4
    /sources/gui/qml/pages/disinfect/DisinfectStack.qml
    • -5
    • +2
    /sources/gui/qml/pages/settings/SettingsDG.qml
4:59 pm

Dara Navaei

UI-DEN-8495_UI Dev Service Manual Disinfection & Flush
UI-DEN-8495_UI Dev Service Manual Disinfection & Flush