Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add fast-connect substitution. Ignore one more error state. #307

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

jlpouffier
Copy link
Contributor

Hidden SSID

A substitution called hidden_ssid was added for people who have hidden SSID.

Taking control will still be necessary.
I think documentation will have to be updated

Ignored Error state.

If you trigger the VPE but do not talk.
The VPE yields a stt-no-text-recognized.
This PR ignores it for the "Red flashes", so the device silently goes back to idle without flashing.

@jlpouffier jlpouffier marked this pull request as ready for review January 23, 2025 09:24
Copy link

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.

@jesserockz
Copy link
Member

Substitutions are not required for this kind of thing.

Once someone has taken control, they can add this to the minimal yaml to achieve the same

wifi:
  fast_connect: true

It will merge with the wifi configuration.

@jlpouffier
Copy link
Contributor Author

I think the substitution is better (really a personal opinion) because we can comment on it and guide the user to turn something to true.

But yeah. 100% agreed it's literally faster your way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants