Try to remove references to spelled out "reverse osmosis" like this (throughout all RO classes). I would rephrase this as "Dialin API sub-class for RO conductivity sensor data access and override commands."
Move the maximum, minimum, and resolution values for both potassium and calcium, as well as the comments in Variables.qml for now, but leave the //TODO comments here.
Please ensure that all ConnectWifi and DisconnectWifi (enum, function, class, function, error string, macro usage, etc.) functions use the 'wifi' as a prefix, like the others, to maintain consistency. Like: ...WifiList... ...WifiInfo... ...WifiConnect... ...WifiDisconnect...
Please,
If it starts with wifi, then use non-capital 'w', like properties.
If in the middle, then use capital 'W'
This change needs to be done in the entire Device module (MVC,qml) .
As we talked, there is only one screen for this state "for now". Since this aligns with the TD Op_Mode states, I believe it is more consistent and future-proof to keep the stack, although it is only one screen, "for now". So, please revert the stack and remove the unused screen, leaving the PreTreatmentCreate.qml. This way, if we need any additional screens later, we have the foundation in place.
