edwardtfn / HomeAssistant-Config

Edward Firmo's Home Assistant config files
MIT License
22 stars 7 forks source link

Inform the user that 1h delay exists after backup is made before starting the updates #11

Open devtekve opened 2 months ago

devtekve commented 2 months ago

Sorry I hit enter and early sent this without writing the description..

My proposal

Add to the description of the "full backup" toggle that a 1h delay will happen after the backup is created.

Why?

Some of us want to test the automation before forgetting it's doing magic for us, in my case, I had it configured nicely but I wasn't expecting it to be delayed 1h because of the full backup. Which is not a problem at all on a day to day basis, but because I wanted to manually test-run it first and check if everything was going according to plan, I was sitting there trying to figure out what did I misconfigure because nothing was being updated... So probably the description hint would've helped me realize it quicker.

Here's the place where the delay is defined

edwardtfn commented 2 months ago

In fact, some backup solutions will provide a sensor indicating when the backup finished. Maybe we should use the 1h only when no sensor is selected. I will look at this.

eimirae commented 3 weeks ago

+1, I was just getting into debugging exactly what was happening and why my updates were not starting within the 30 min schedule I had set up, and this post helped me figure out what the problem was.

edwardtfn commented 3 weeks ago

I think that delay should be removed. I'm quite busy those days, but will try to look at this next week.

edwardtfn commented 2 weeks ago

I've updated the Blueprint to be compatible to the latest Home Assistant (mainly the replacement of Services by Actions, but also introduced the collapsible sections to make it cleaner) and HACS changes (which now supports update entities. I've also added this timeout as an input, so it is more clear to users (and also selectable). Please let me know your thoughts.