Update sensor.py to include optional Air Quality Sensors

[mcattle]

If the Kidde Smoke + CO detector also includes the Air Quality Monitor feature, additional entities will be created to measure the various air quality values. These measurement entities also have a "Status" attribute that includes the Kidde summary of the value as "Good", "Moderate", and such.

[dandanio]

That would be an awesome update.

[GiveMeTheBits]

Adding my vote to get this approved. the IAQ data is what I am most interested in gathering via Home Assistant.

[alloveru]

great addition!
good work to you both!

[fixthisbuildthat]

Also would love the temp and humidity data. whole reason I wanted to add these to HA. Thanks!

[mcattle]

@865charlesw Would you mind reviewing this PR and merging it?

[emmarichardson]

Did you add more changes in your repo @mcattle - I tried switching out the sensor.py file with the one from your repo but HomeAssistant errored out on restart (on that integration only)...said it couldn't load the config so I am guessing that maybe you have added some more changes in your repo and I would have to install from scratch with yours?

[mcattle]

Did you add more changes in your repo @mcattle - I tried switching out the sensor.py file with the one from your repo but HomeAssistant errored out on restart (on that integration only)...said it couldn't load the config so I am guessing that maybe you have added some more changes in your repo and I would have to install from scratch with yours?

No, I'm pretty sure my only changes are to that one file. Could you post your error log so that I may take a look?

[emmarichardson]

Sorry to post that way but was having issues copying and pasting

[mcattle]

@emmarichardson I think you copied too much into that file. Your sensor.py should contain just this: sensor.py

[tankdeer]

@mcattle not trying to volunteer you, but have you considered forking this repo and running with it a little more? It seems like @865charlesw might be MIA, and hasn't shown any activity since mid November. I know there are plenty of people interested in this

[mcattle]

@mcattle not trying to volunteer you, but have you considered forking this repo and running with it a little more? It seems like @865charlesw might be MIA, and hasn't shown any activity since mid November. I know there are plenty of people interested in this

I'm not a Python dev, though, I'm a C# dev. I can fiddle enough to add a couple sensors, but I can't add support for firmware upgrade detection and installation, which is a feature I also want. @865charlesw did mention that his last push was "before he left", but I don't know to where or for how long. I'm hoping he'll turn up soon.

[tankdeer]

@mcattle not trying to volunteer you, but have you considered forking this repo and running with it a little more? It seems like @865charlesw might be MIA, and hasn't shown any activity since mid November. I know there are plenty of people interested in this

I'm not a Python dev, though, I'm a C# dev. I can fiddle enough to add a couple sensors, but I can't add support for firmware upgrade detection and installation, which is a feature I also want. @865charlesw did mention that his last push was "before he left", but I don't know to where or for how long. I'm hoping he'll turn up soon.

Entirely reasonable. I am not great with Python myself. Fingers crossed he shows up and this integration is not DOA

[emmarichardson]

@mcattle I literally did a wget on the file from your repo after moving the old one...

[mcattle]

@mcattle I literally did a wget on the file from your repo after moving the old one...

It appears that you did a wget of the HTML page, as opposed to the raw file.

[mcattle]

@865charlesw Any chance on getting this PR merged?