NebraLtd / hm-diag

Helium Miner Diagnostics
https://nebra.io/hnt
MIT License
21 stars 25 forks source link

Only show LTE for LTE enabled variants. #101

Closed robputt closed 3 years ago

robputt commented 3 years ago

LTE detected shows even for non-outdoor units. I'm wondering whether we should add a field to the hardware variants of "LTE Support" or something to that effect? So that we can automatically determine whether it should even bother to check for LTE? - https://github.com/NebraLtd/hm-diag/pull/92#issuecomment-898944681

shawaj commented 3 years ago

guess we will need to add celullar to tests as well https://github.com/NebraLtd/hm-diag/blob/master/hw_diag/tests/fixtures/diagnostic_results.py