pinkywafer / Calendarific

Calendarific holiday sensor for Home Assistant
MIT License
19 stars 7 forks source link

Add OptionsFlow and Update ConfigFlow #34

Closed Snuffy2 closed 2 weeks ago

Snuffy2 commented 2 years ago

I was looking for a way to change the icons and soon days without removing and recreating the sensor.

pinkywafer commented 1 year ago

looks good. Thanks. I'll have a good look over this when I get chance, but I'm happy to merge in principle

pinkywafer commented 1 year ago

Please do not perform code style changes / cleanup as part of a PR adding something as I have commented on your other PR.

Also, please remove the date format options - This config option is going to be removed soon as Home assistant does not respect it - it sees a date and displays it as such.

Snuffy2 commented 1 year ago

I've removed the formatting changes and commented out the Date Format from the ConfigFlow and OptionsFlow.