-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
ATOM Echo: replace steps by wizard #39294
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
Conversation
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughThe setup instructions for the ATOM Echo device in Home Assistant were updated to describe a wizard-based customization process, clarify device connection steps, generalize the wake word example, and correct minor textual details regarding configuration navigation and terminology. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant HomeAssistant
participant ATOM_Echo
User->>HomeAssistant: Start device setup
HomeAssistant->>User: Launch customization wizard
User->>HomeAssistant: Define wake word and choose voice
User->>HomeAssistant: Select "Done"
HomeAssistant->>ATOM_Echo: Connect over Wi-Fi
ATOM_Echo-->>HomeAssistant: Confirm connection
HomeAssistant->>User: Display confirmation and next steps
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
source/voice_control/thirteen-usd-voice-remote.markdown (2)
73-73
: Fix grammar in wizard description.The sentence “This starts the a wizard to customize the your voice assistant.” contains redundant determiners. It should read:
6. This starts a wizard to customize your voice assistant.🧰 Tools
🪛 LanguageTool
[grammar] ~73-~73: Two determiners in a row. Choose either “the” or “a”.
Context: ...sp32-atom-flash-07.png) 6. This starts the a wizard to customize the your voice assi...(DT_DT)
[grammar] ~73-~73: A determiner cannot be combined with a possessive pronoun. Did you mean simply “the” or “your”?
Context: .... This starts the a wizard to customize the your voice assistant. - Follow the wizard...(A_MY)
81-81
: Improve example emphasis.For consistency with other docs, consider using backticks or italics for the example wake word. E.g.:
1. Say the wake word you configured. For example, use _OK, Nabu_.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting
⛔ Files ignored due to path filters (1)
source/images/assist/esp32-atom_silence_detection_01.png
is excluded by!**/*.png
📒 Files selected for processing (1)
source/voice_control/thirteen-usd-voice-remote.markdown
(2 hunks)
🧰 Additional context used
🪛 LanguageTool
source/voice_control/thirteen-usd-voice-remote.markdown
[grammar] ~73-~73: Two determiners in a row. Choose either “the” or “a”.
Context: ...sp32-atom-flash-07.png) 6. This starts the a wizard to customize the your voice assi...
(DT_DT)
[grammar] ~73-~73: A determiner cannot be combined with a possessive pronoun. Did you mean simply “the” or “your”?
Context: .... This starts the a wizard to customize the your voice assistant. - Follow the wizard...
(A_MY)
🔇 Additional comments (2)
source/voice_control/thirteen-usd-voice-remote.markdown (2)
74-75
: Approve wizard steps formatting.The new instructions to follow the wizard steps and select Done are clear and consistent with the updated setup flow.
93-94
: Verify integration path and setting names.Please confirm that the macro
{% my integrations title="**Settings** > **Devices & services**" %}
and the label Finished speaking detection match exactly what users see in the UI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @c0ffeeca7 👍
../Frenck
Proposed change
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit