CestLaGalere / mobilealerts

Retrieves sensor data for Mobile Alerts sensors for Home Assistant
Apache License 2.0
15 stars 6 forks source link

feat: add sensor Entities and descriptors #3

Closed petrleocompel closed 1 year ago

petrleocompel commented 1 year ago

Now HA have temperature and humidity as graphs. Same could be done with others. It was drunk quick fix during garage session. For @vasule

I plan to make a bigger clean up later. (old typings, split in files etc...)

CestLaGalere commented 1 year ago

I have merged this and it now gives an unknown error when I try and update the integration in HACS (should add my thanks for contributing!)

petrleocompel commented 1 year ago

The reason will be pr #4 ->

In configuration.yml you have to correct the "mobilealerts" to "mobile_alerts". Later during any display it will use correct domain and icon from "home assistant" brands

CestLaGalere commented 1 year ago

ah - my bad for not reading this thoroughly enough! Many thanks - do you want permissions to merge (I assume that's admin)?

petrleocompel commented 1 year ago

That is up to you. Currently there is more to enable. Github Actions would check for validity of manifest and basic installation. Later can do automatic tests.