search

finity69x2 / nws_alerts

An updated version of the nws_alerts custom integration for Home Assistant
86 stars 29 forks source link

Add Config Flow option #1

Closed firstof9 closed 4 years ago

firstof9 commented 4 years ago
finity69x2 commented 4 years ago

Great! Now I'll have to figure out how all of the config flow stuff works so I can maintain that too! ;)

Thanks for doing all of that. I definitely had no plans for it.

I hope it's all correct because if it's not I wouldn't have any idea (yet...?) on how to fix it. :)

firstof9 commented 4 years ago

I've tested the config flow bits myself, not errors and the sensors are created. Forgot to put that I added the attribution attribute to the sensor as well, the "Data provided by ..." bits. The sensor is re-configurable from integrations as well via the "options", so it can be changed.

Really the config flow bits shouldn't break unless you need to add more options. I'd be happy to help if anything breaks as well.

finity69x2 commented 4 years ago

I merged this but I jumped too soon.

I had made some changes and accidentally uploaded my test file instead of the correct file a couple of days ago. I assumed you made changes to the latest version.

Now I've got a few corrections to make but they should be minor.