Add alarm support notes for Sonos#44476
Open
PeteRager wants to merge 3 commits intohome-assistant:currentfrom
Open
Add alarm support notes for Sonos#44476PeteRager wants to merge 3 commits intohome-assistant:currentfrom
PeteRager wants to merge 3 commits intohome-assistant:currentfrom
Conversation
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Sonos integration documentation to clarify limitations when modifying multiple Sonos alarms in quick succession, and how to avoid missed alarm updates.
Changes:
- Rewords the existing alarm switch description for clarity and consistency.
- Adds guidance about Sonos alarm list synchronization limitations when updating multiple alarms rapidly.
- Adds a YAML example showing inserting delays between alarm updates.
Comment on lines
+79
to
+87
| When enabling, disabling, or updating multiple Sonos alarms in a single automation or script, you may notice that not all alarms update as expected. This is a limitation of how Sonos synchronizes its alarm list across all speakers. If multiple alarm updates are sent rapidly (for example, by toggling several alarm switches at once), some changes may be lost or overwritten before they are fully synchronized between speakers. | ||
|
|
||
| To ensure reliable updates, add a wait between each alarm update in your automation. A 1 second wait works well as a starting point, but you may need to increase it based on your setup. For example, add a `delay` action between each `switch.turn_on`, `switch.turn_off`, or `sonos.update_alarm` action call. This gives Sonos time to synchronize the alarm list before the next change. | ||
|
|
||
| _Example:_ | ||
|
|
||
| ```yaml | ||
| sequence: | ||
| - action: switch.turn_on |
There was a problem hiding this comment.
The text says this applies to an “automation or script,” but the YAML example is structured as a script sequence: snippet. To avoid copy/paste confusion, either adjust the wording to explicitly say the example is for a script sequence, or provide an automation-style example (using actions:) as well.
Contributor
Author
There was a problem hiding this comment.
Changed to be a sample automations
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed change
Type of change
currentbranch).currentbranch).nextbranch).nextbranch).Additional information
Checklist
currentbranch.nextbranch.