RESOLVED

Done

1 - I mean that timer is not only for handling our implementation it may have another purpose which we are stopping by killing the only timer it has.
2 - in your

timerEvent {
    ...
    if( ! _timerIsActive ) return;
    ...
}

Since any object has only one timer I have a feeling that by killing the object's only timer, then any refreshing or chores for any kind of signal/slot maintenance will be stoped.

That's the intention - to stop the timer. Since the UI is continually trying to reconnect to the last paired device while it is off, if a user goes into the bluetooth settings page and hits "Scan for devices", the timer is stopped during the scan to avoid conflicts between the timer and the scan.

I prefer to have a boolean flag to return from the virtual method in case we don't need it to handle the functionality we put there.

This doesn't make sense to me, can you clarify?

RESOLVED

I am not sure killing the QObject timer would be a good idea, but also I couldn't find any recommendation not to do it.
Since any object has only one timer I have a feeling that by killing the object's only timer, then any refreshing or chores for any kind of signal/slot maintenance will be stoped.

I prefer to have a boolean flag to return from the virtual method in case we don't need it to handle the functionality we put there.

When we add the correct object dictionary in the names.py then sys/v&v/... can use it and don't need the object name anymore to do so.
Not 100% agree.
Also, we need to have a consistent naming for the currently existing objectName in the code, I'll later provide a standard.

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

Please note that the code review has to be automatic, by making it manual and mixed up with other features, this may happen.
RESOLVED

RESOLVED

It's not the components themselves that do what you want it's how you call and manage it and having another component was unnecessary.
RESOLVED regardless.

RESOLVED

RESOLVED

RESOLVED

Done

Done

RESOLVED

So we may need to be careful when we are adding the WiFi driver in here later.
RESOLVED.

Perfect,
RESOLVED.

We need to have a folder next to the application for all the settings since it is not related to the treatment it is only settings configuration files.
Since the SD card might be damaged and we still need to store settings.
Please change it to refer to the application location and name the folder as configuration or settings.

Oh I see.
RESOLVED

It's in the UX design here: https://app.zeplin.io/project/5e160353a7c41a9404596a70/screen/5e5060d3f75589612aee8b68

See response above

See response above