uvjim / linksys_velop

Home Assistant integration for the Linksys Velop
MIT License
50 stars 7 forks source link

[Feature Request] Restart whole system button #462

Closed GSzabados closed 6 months ago

GSzabados commented 6 months ago

Could be a simple reboot button added for the mesh/primary node?

I have to restart irregularly my whole setup, and doing it with a service call is rather cumbersome, as I never know what the entity or device called, as it not fetches it for the service and I never know how to set the primary switch there and so on. So it is always a trial and error first, and takes a few minutes. A single button would streamline this. Like a Whole Mesh Reboot under the primary node.

Thanks!

GSzabados commented 6 months ago

You could name it: Reboot whole mesh system!

uvjim commented 6 months ago

You'll be pleased to know that I pre-empted the request and checked in a solution earlier today.

I'll release another beta later today.

GSzabados commented 6 months ago

Thanks! I really appreciate it!

uvjim commented 6 months ago

Should be available in 2024.5.1b1 now. You may want to check the README for how to enable it.

GSzabados commented 6 months ago

Thank you very much! It works flawlessly. I just tested it. 😄 Additionally I would update the readme on the top where it says it does not provide a button for this, saying as default, but you can enable it.

This part:

N.B. There is deliberately no button provided to restart the Primary node. Restarting the Primary node will cause all nodes in the Mesh to reboot and I consider this to be quite a destructive action. There is no confirmation in the HASS UI when a button is pressed so there is no time to warn anyone. If you'd like to reboot the Primary you node you must use the service with the is_primary flag provided.

uvjim commented 6 months ago

I'll update that before release - completely forgot I'd put that statement in there. 😂

GSzabados commented 6 months ago

I see some new error message in the beta. I will open a new issue for that.

uvjim commented 6 months ago

Implemented in #461