UI-DEN-704

RESOLVED

RESOLVED

RESOLVED

RESOLVED

UI-DEN-704_HomeScreen
UI-DEN-704_HomeScreen
RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

I meant : "Can Interface has been setup" like that I have as : "candump launched"

I meant :
"Can Interface has been setup"

like that I have as :
"candump launched"

For the two eChlid_UI and eChlid_HD which are for UI in UI I prefer to keep them simple to use and read.

For the two eChlid_UI and eChlid_HD which are for UI in UI I prefer to keep them simple to use and read.

it has been addresses : *http://dvm-linux02:8060/project/UI-DEN-1090
Behrouz 0x20 is a channel ID that is from UI to HD. This identifier does not reflect that. Please reconsider renaming.

Behrouz 0x20 is a channel ID that is from UI to HD. This identifier does not reflect that. Please reconsider renaming.

Same comment as before. Verb.

Same comment as before. Verb.

Recommend using a more descriptive naming. 0x020 is an input channel to UI, i.e., HD2UI. Similar is 0x100, it is an output channel.From UI to HD.

Recommend using a more descriptive naming. 0x020 is an input channel to UI, i.e., HD2UI. Similar is 0x100, it is an output channel.From UI to HD.

Oh I changed it to initConnections() in new pushes as part of DEN-1090 : DEN-1090-1 when I saw your comment.

Oh I changed it to initConnections() in new pushes as part of DEN-1090 : DEN-1090-1 when I saw your comment.

It has been done in new codes for DEN-1090 : DEN-1090-1. But as a general comment its my preferred coding style for readability and its not our rule or requirement.

It has been done in new codes for DEN-1090 : DEN-1090-1.
But as a general comment its my preferred coding style for readability and its not our rule or requirement.

Can Interface has its own error information. What I added here is an extra information.

Can Interface has its own error information.
What I added here is an extra information.

I was going to use connect but the parent has one then it will override it which is wrong. So I changed it to connection.

I was going to use connect but the parent has one then it will override it which is wrong.
So I changed it to connection.

Line up onRead() slot with frameReceived()... e.g. onCANFrameReceived()

Line up onRead() slot with frameReceived()... e.g. onCANFrameReceived()

Device creation is too generic, make it more specific "Can interface cannot be initialize"

Device creation is too generic, make it more specific "Can interface cannot be initialize"

Same as before. Methods must be verbs.

Same as before. Methods must be verbs.

Add commented line with the name of the script. Don't we need headers for these scripts?

Add commented line with the name of the script. Don't we need headers for these scripts?

The message should be more along the lines that CAN interface is running...

The message should be more along the lines that CAN interface is running...

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.