Skip to content

[automower] Update README.md#20666

Open
mwellmeyer wants to merge 1 commit intoopenhab:mainfrom
mwellmeyer:patch-1
Open

[automower] Update README.md#20666
mwellmeyer wants to merge 1 commit intoopenhab:mainfrom
mwellmeyer:patch-1

Conversation

@mwellmeyer
Copy link
Copy Markdown

fix park-until-* command in documentation and example

fix park-until-* command in documentation and example

Signed-off-by: Matthias Wellmeyer <m.wellmeyer@web.de>
@jlaur jlaur changed the title Update README.md [automower] Update README.md May 5, 2026
@lsiepel lsiepel requested review from MikeTheTux and Copilot May 5, 2026 09:50
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Automower binding README to correct the documented channel IDs for the “park until …” command channels so they match the binding’s actual hyphenated channel IDs.

Changes:

  • Fix command channel IDs from park_until_* to park-until-* in the command-channel table.
  • Fix the corresponding example *.items channel references to use park-until-*.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -255,8 +255,8 @@ Number Automower_Command_Start "Start mowing for duration [%d min
Switch Automower_Command_Resume "Resume the schedule" { channel="automower:automower:mybridge:12345678-1234-12ab-1234-123456abcdef:command#resume_schedule" }
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also correct the others while you are at it? All underscores got replaced by hyphens AFAICT

Suggested change
Switch Automower_Command_Resume "Resume the schedule" { channel="automower:automower:mybridge:12345678-1234-12ab-1234-123456abcdef:command#resume_schedule" }
Switch Automower_Command_Resume "Resume the schedule" { channel="automower:automower:mybridge:12345678-1234-12ab-1234-123456abcdef:command#resume-schedule" }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

start_in_workarea

Copy link
Copy Markdown
Contributor

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants