Revert "Add restart button to home-assistant-voice.yaml"

[balloob]

There is no normal operations use case for restarting the Voice PE. We shouldn't add features "because we can".

Reverts #273

[github-actions]

Firmware built successfully! 🎉

Download and extract the firmware to install with https://web.esphome.io

Make sure to choose home-assistant-voice-esp32s3.factory.bin.

[the-mentor]

There are a bunch of use cases that require device reboot

Lets say I change a dhcp reservation and want to remotely apply the change to the device.

Or if I change dns settings on my router and I want the device to get them.

I see what you are saying about adding features just because you can
I just don't think a reboot is not a resonable option to have

Here is the original issue
#270

[tbrasser]

Also its disabled by default, so why revert?

Usecase: have single ssid "mesh" (wired backbone), esp32's don't seem to roam/connect to closest AP when something happens to them (e.g. AP firmware update doesn't connect back to it after coming online again), remote reboot is needed. (or does this wifi config have fast bss / roaming enabled? That should also work for this usecase if the network is configured as such)

[RocketRammer]

There are a bunch of use cases that require device reboot

Lets say I change a dhcp reservation and want to remotely apply the change to the device.

Or if I change dns settings on my router and I want the device to get them.

I see what you are saying about adding features just because you can I just don't think a reboot is not a resonable option to have

Here is the original issue #270

Yep this. My DNS fell over the other other. Fixed it. Then noticed both my Voice PE's were sat with a red ring of death. Had to manually unplug them like some 90's hardware.

Crazy not to add this.

I understand the approach of not doing things just because you can, however I'd challenge that with what harm does this bring? Zero. If people don't need it, they won't use it, but it's there for the people that do.